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

Readme fix #2 #658

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Readme fix #2 #658

wants to merge 2 commits into from

Conversation

4S1ght
Copy link
Contributor

@4S1ght 4S1ght commented Jul 7, 2024

I did some more in-depth reading and found a few more grammatical mistakes in the readme.
Some wrongly formulated sentences and inconsistent casing.

I'm not quite sure how to understand "instance modules instance" term appearing frequently across the documentation, such as on lines 688, 692 & 728.

Other than that, most glaring issues should be fixed by now.
If some changes I made are wrong then feel free to correct my mistakes.

Lastly, I'm sorry I couldn't update the docs website as well, but I found myself in a little bit of a squeeze lately.
I do hope that I helped at least a bit though.
@4S1ght
Copy link
Contributor Author

4S1ght commented Jul 8, 2024 •
edited
Loading

Just noticed some signs of me fiddling with the snippets came through that I didn't notice before when committing on line 208 - Sorry about that. I guess I'm just so used to seeing costs everywhere that I didn't notice it there.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
1 participant
@4S1ght