Skip to content

Idiomatic: Various tweaks and cleanup to naming conventions section#3164

Open
randomPoison wants to merge 2 commits intogoogle:mainfrom
randomPoison:naming-conventions-tweaks
Open

Idiomatic: Various tweaks and cleanup to naming conventions section#3164
randomPoison wants to merge 2 commits intogoogle:mainfrom
randomPoison:naming-conventions-tweaks

Conversation

@randomPoison
Copy link
Copy Markdown
Contributor

Tweaks and feedback to the "Naming Conventions" section of Idiomatic based on my experience teaching the class, mostly focusing on making the slides more concise. Related to #3157, which also attempts to trim the section by removing some slides that I don't think are necessary. As noted in that PR, I think the section is overly long and would benefit from being a quicker tour of naming conventions, without needing to spend a lot of time on any of the slides.

Copy link
Copy Markdown
Collaborator

@mgeisler mgeisler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for condensing this!

@mgeisler mgeisler enabled auto-merge (squash) April 20, 2026 21:51
auto-merge was automatically disabled April 21, 2026 20:01

Head branch was pushed to by a user without write access

@randomPoison randomPoison requested a review from mgeisler April 21, 2026 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants