Skip to content

Update comment from "Good" to "Bad" #843

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Conversation

dane
Copy link

@dane dane commented Jul 19, 2024

👋 I know you're not going to merge this PR. It seemed like the easiest way to communicate which line/file had the typo. Cheers!

@vapier vapier added the lang:go The Go language label Dec 3, 2024
@matttproud
Copy link
Member

The fix for this has been committed in the upstream documentation (source of truth) and will propagate into the public style guide documentation on the next export. Thank you for flagging the issue! I will close this pull request due to the SoT change.

@matttproud
Copy link
Member

Changed in upstream.

@matttproud matttproud closed this May 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang:go The Go language
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants