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.

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

Minor fixes the README #70

Merged
merged 1 commit into from
Oct 24, 2023
Merged

Minor fixes the README #70

merged 1 commit into from
Oct 24, 2023

Conversation

tabrownies
Copy link
Contributor

Fixed spelling, grammar, and phrasing. I did my best to not change the meaning of the text in the README and just fix the problems. Because of that, some grammar issues remain,but changing them would potentially change the meaning of the paragraph, so they were not changed.

Fixed spelling, grammar, and phrasing. I did my best to *not* change 
the meaning of the text in the README and just fix the problems. 
Because of that, some grammar issues remain,but changing them would 
potentially change the meaning of the paragraph, so they were not 
changed.
Copy link
Contributor

@Xinchunran Xinchunran 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 spelling improvement!

@tabrownies
Copy link
Contributor Author

I can't merge the pull request because I don't have sufficient privileges. Could one of you do that? @wwwwww1020, @colinlaoaa, or @Xinchunran?
Also, I made an issue to go along with this pull request that we can link to it, which I also don't have permission to do.

@wwwwww1020 wwwwww1020 merged commit 50b7ae8 into AI-Citizen:main Oct 24, 2023
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.

None yet

4 participants