Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update docs for higher accuracy and quality of information #648

Merged
merged 5 commits into from
Nov 26, 2024
Merged

Update docs for higher accuracy and quality of information #648

merged 5 commits into from
Nov 26, 2024

Conversation

intls
Copy link
Contributor

@intls intls commented Nov 25, 2024

Problem

Typos and grammar and spelling errors around docs files.

Summary of Changes

Fixed critical inaccuracies and enhanced readability of the documentation to ensure better understanding by developers.

Fixes #

@intls intls requested a review from ZYJLiu as a code owner November 25, 2024 22:29
nickfrosty
nickfrosty requested changes Nov 25, 2024
View reviewed changes
Copy link
Collaborator

@nickfrosty nickfrosty 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 fixing all these grammar issues. Can you run the prettier formatting to satisfy the prettier action check?

@nickfrosty nickfrosty added the prettier CI: auto apply the local prettier config to a PR label Nov 26, 2024
@nickfrosty nickfrosty merged commit 0a6a045 into solana-foundation:main Nov 26, 2024
2 of 4 checks passed
@intls
Copy link
Contributor Author

intls commented Nov 27, 2024

Thanks for fixing all these grammar issues. Can you run the prettier formatting to satisfy the prettier action check?

Thanks!Sure I will make it asap!

@intls
Copy link
Contributor Author

intls commented Nov 29, 2024

Thanks for fixing all these grammar issues. Can you run the prettier formatting to satisfy the prettier action check?

@nickfrosty HEY!! Prettier has been applied to the specified files, but no changes were necessary as they were already correctly formatted. Let me know if there's anything else I should address!

@nickfrosty
Copy link
Collaborator

Thanks! I went ahead and ran the prettier myself and the PR was already merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nickfrosty nickfrosty nickfrosty approved these changes

@ZYJLiu ZYJLiu Awaiting requested review from ZYJLiu ZYJLiu is a code owner

Assignees
No one assigned
Labels
prettier CI: auto apply the local prettier config to a PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@intls @nickfrosty