Skip to content
New issue

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

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

Already on GitHub? # to your account

Minor improvements to documentation files #3061

Merged
merged 6 commits into from
Nov 19, 2023
Merged

Minor improvements to documentation files #3061

merged 6 commits into from
Nov 19, 2023

Conversation

cvyl
Copy link
Contributor

@cvyl cvyl commented Nov 6, 2023

Grammatical fixes.

@cvyl cvyl changed the title Patch 1 Minor improvements to documentation files Nov 6, 2023
Copy link
Owner

@Bionus Bionus 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 that! I'm not a native speaker and seems like it really shows from the number of small issues everywhere 😅

I just have two small points/questions, if you don't have time feel free to tell me and I'll update the PR directly on Tuesday when my internet is back!

Thanks again 🙏

docs/docs/filename.md Outdated Show resolved Hide resolved
docs/docs/filename.md Show resolved Hide resolved
docs/docs/index.md Outdated Show resolved Hide resolved
Copy link
Contributor Author

@cvyl cvyl left a comment

Choose a reason for hiding this comment

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

Let me know what you think of these changes 😊

docs/docs/index.md Show resolved Hide resolved
@cvyl
Copy link
Contributor Author

cvyl commented Nov 10, 2023

I am unsure how GitHub handles people changing their usernames, but I think it all still works! (I am new to using GitHub after almost five years of inactivity)

Copy link
Contributor Author

@cvyl cvyl left a comment

Choose a reason for hiding this comment

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

I forgot to review the files but I am sure this is correct.

Copy link
Owner

@Bionus Bionus left a comment

Choose a reason for hiding this comment

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

LGTM, thanks again! 👍

@Bionus Bionus merged commit 9106877 into Bionus:master Nov 19, 2023
1 check passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants