Skip to content
This repository has been archived by the owner on Sep 24, 2022. It is now read-only.

Update about-quickstart.md #118

Merged
merged 2 commits into from
May 14, 2020
Merged

Conversation

shreyas1599
Copy link
Contributor

Made the changes suggested in #38. Also, a couple of changes I'd like to make as well, but need an opinion before doing so. A few sentences like the following use will. I suggest removing or replacing the word. For example:

It should explain the basic operations that most users will want to perform on the site, or the operations they will perform when they use the API.

It should explain the basic operations that most users perform on the site, or the operations they perform when they use the API.

Start by telling users what the quickstart will do

Start by telling users what the quickstart does.

Basically, a few tense changes. There are a couple more wills in the document. I'll make these changes if(/when) a maintainer accepts them.

Copy link
Contributor

@jaredmorgs jaredmorgs left a comment

Choose a reason for hiding this comment

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

These changes look good to me.

@shreyas1599
Copy link
Contributor Author

@jaredmorgs Can I go ahead and add a commit for the other changes I suggested above?

@jaredmorgs
Copy link
Contributor

jaredmorgs commented May 12, 2020 via email

@jaredmorgs
Copy link
Contributor

Just that minor change in the last sentence looks good. All other changes look good. I'll merge this in now.

@jaredmorgs jaredmorgs merged commit 26cd1ed into thegooddocsproject:master May 14, 2020
camerons pushed a commit to camerons/templates that referenced this pull request Feb 27, 2021
* Update about-quickstart.md

* update: re-wording sentences
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants