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

Update README.mkd #815

Merged
merged 5 commits into from
Apr 26, 2022
Merged

Update README.mkd #815

merged 5 commits into from
Apr 26, 2022

Conversation

NGWi
Copy link
Contributor

@NGWi NGWi commented Apr 26, 2022

Corrected typos and polished some of the grammar when it at least didn't make the doc harder to understand.
I also added some instructions in section 8 and made it easier to follow (I believe).

NGWi added 2 commits April 26, 2022 01:24
Corrected typos and polished some of the grammar when it at least didn't make the doc harder to understand.
I also added some instructions in section 8 and made it easier to follow (I believe).
@NGWi NGWi marked this pull request as ready for review April 26, 2022 05:47
Updated formatting (`` instead of '') for what I added to Section 8, so more items are consistently highlighted.
README.mkd Outdated Show resolved Hide resolved
README.mkd Outdated Show resolved Hide resolved
@nsantacruz
Copy link
Contributor

Thanks! I fixed some typos I found but overall seems like an improvement.

@nsantacruz nsantacruz merged commit 6b4b9bc into Sefaria:master Apr 26, 2022
@NGWi
Copy link
Contributor Author

NGWi commented Apr 26, 2022 via email

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