Skip to content

docs: improve grammar, clarity, and code examples and API docs#548

Merged
stevenh merged 3 commits intorobertkrimen:masterfrom
dev-kas:master
Jun 13, 2025
Merged

docs: improve grammar, clarity, and code examples and API docs#548
stevenh merged 3 commits intorobertkrimen:masterfrom
dev-kas:master

Conversation

@dev-kas
Copy link
Copy Markdown
Contributor

@dev-kas dev-kas commented Jun 9, 2025

This PR updates the README and related docs to fix grammar issues, clarify language, and correct minor code example errors. The changes improves readability and consistency without changing any functionality.

Improvements:

  • Fixed punctuation and grammar across multiple sections
  • Rewrote informal or unclear phrases for increased clarity
  • Corrected code examples (e.g., missing fmt.Printf format strings)
  • Improved consistency in function descriptions

These updates make the docs more professional and accessible for both new and experienced.

- fixed grammar and punctuation issues
- improved clarity and consistency in phrases
- corrected minor code example errors
Copy link
Copy Markdown
Collaborator

@stevenh stevenh 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 the PR, no need for commas after e.g. when using real english 😃

Comment thread README.md
Comment thread README.md Outdated
Comment thread README.md Outdated
Comment thread README.md Outdated
- Remove commas after 'e.g.' per maintainer feedback
- Improve error handling in example code
- Maintain consistency in documentation style

Addresses feedback from PR robertkrimen#548
@dev-kas
Copy link
Copy Markdown
Contributor Author

dev-kas commented Jun 10, 2025

Thanks for the feedback! Ive removed the commas after "e.g." as suggested. I also enhanced the error handling in the first code example for better clarity.

Let me know if there's anything else you'd like me to adjust.

Copy link
Copy Markdown
Collaborator

@stevenh stevenh left a comment

Choose a reason for hiding this comment

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

Just a missing full stop and then good to go, thanks for the quick turn around.

Comment thread README.md Outdated
Adds missing period at the end of the sentence in the Export function's
documentation to maintain consistent punctuation style.
@dev-kas
Copy link
Copy Markdown
Contributor Author

dev-kas commented Jun 10, 2025

I've added the missing period to the Export function's documentation. The Pr should now be ready for your review.

@stevenh stevenh merged commit 3ca7298 into robertkrimen:master Jun 13, 2025
2 checks passed
@stevenh
Copy link
Copy Markdown
Collaborator

stevenh commented Jun 13, 2025

Thanks for the PR @dev-kas

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