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

Fixed Typos in Docs #414

Merged
merged 2 commits into from
Aug 18, 2022
Merged

Fixed Typos in Docs #414

merged 2 commits into from
Aug 18, 2022

Conversation

10kartik
Copy link
Contributor

Fixed a few grammatical and sentence construction mistakes from docs.

Copy link
Contributor

@jogo jogo left a comment

Choose a reason for hiding this comment

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

Thank you for the cleanup! I have one very tiny question out of curiosity, will merge afterwards.

to quit your interpreter. It can allow you to see what's happen if your server
is abruptly closed. Below is an by example.
to quit your interpreter. It can allow you to see what happens if your server
is abruptly closed. Below is a by-example.
Copy link
Contributor

Choose a reason for hiding this comment

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

Out of curiosity can you point me to the grammar guideline for the dash in 'by-example' I haven't personally seen that one before.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah, sorry! There shouldn't have been '-' but 'by example' instead or 'an example' as suggested by @jparise below.

docs/getting_started.rst Outdated Show resolved Hide resolved
@jogo jogo merged commit f34ac28 into pinterest:master Aug 18, 2022
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.

3 participants