Skip to content

Idiomatic: Various minor tweaks to doc comments section#3162

Merged
mgeisler merged 3 commits intogoogle:mainfrom
randomPoison:doc-comments-section-cleanup
Apr 28, 2026
Merged

Idiomatic: Various minor tweaks to doc comments section#3162
mgeisler merged 3 commits intogoogle:mainfrom
randomPoison:doc-comments-section-cleanup

Conversation

@randomPoison
Copy link
Copy Markdown
Contributor

Various minor tweaks to the "Meaningful Doc Comments" section of Idiomatic based on my experience teaching the class for the first time.

  • Fix a few formatting issues in speaker notes that caused the markdown to not render cleanly.
  • Trim a few speaker notes that I think are redundant.
  • anatomy-of-a-doc-comment.md - Reorder speaker note so the Q&A about panics is organized under the section about panics.
  • what-why-not-how-where.md - Remove the sort_quickly example. This is later used as the example in the exercise, and I think discussing it before the exercise undercuts the value of the exercise.

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

Head branch was pushed to by a user without write access

@randomPoison randomPoison requested a review from mgeisler April 21, 2026 20:06
@mgeisler mgeisler enabled auto-merge (squash) April 28, 2026 16:01
@mgeisler mgeisler merged commit 58f6817 into google:main Apr 28, 2026
37 checks passed
@randomPoison randomPoison deleted the doc-comments-section-cleanup branch April 28, 2026 16:25
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