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

docs: enhance quick start with lessons learned #15900

Merged
merged 3 commits into from
Feb 21, 2017
Merged

Conversation

gu-stav
Copy link
Contributor

@gu-stav gu-stav commented Feb 21, 2017

What does this change?

Enhances quickstart documentation.

What is the value of this and can you measure success?

Better onboarding experience for upcoming people.

@gu-stav gu-stav changed the title docs: enhance quickstart guide with lessons learned docs: enhance quick start with lessons learned Feb 21, 2017
@sndrs
Copy link
Member

sndrs commented Feb 21, 2017

you may need to add your guardian email address to your github account

is that correct?

@gtrufitt
Copy link
Contributor

@sndrs it's one of the things we did though I don't know what would happen if we didn't do it...

From Github docs though

If you set an email address in your local Git configuration, you will need to add it to your account settings in order to connect your commits to your account.

@@ -25,7 +25,7 @@ Follow [this link](https://www.google.co.uk) and enter the relevant search strin

# Local Test Server setup

You can either set up the frontend through a [local install](#local-machine) on your machine or using [Docker](#docker). Before checking out the repository you may need to [add an SSH key to your GitHub account](https://help.github.com/articles/generating-ssh-keys/). Before pushing changes you may need to [create an access token](https://help.github.com/articles/creating-an-access-token-for-command-line-use/).
You can either set up the frontend through a [local install](#local-machine) on your machine or using [Docker](#docker). Before checking out the repository you may need to add your guardian email address to your github account and [add an SSH key to your GitHub account](https://help.github.com/articles/generating-ssh-keys/). Before pushing changes you may need to [create an access token](https://help.github.com/articles/creating-an-access-token-for-command-line-use/). Also make sure, your github account was added to all concerned teams, in order to have full access.
Copy link
Contributor

@TBonnin TBonnin Feb 21, 2017

Choose a reason for hiding this comment

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

Also make sure your github account was added to all concerned teams, in order to have full access

Passive voice can probably be replaced by something more specific like Make sure your dev manager has added you to team X and Y, so you can have access to all necessary repositories

What do you think?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Generally I like the idea, but I wasn't sure, whether this is too guardian specific.

What do you think about a mix: Make sure your dev manager has added you to the necessary teams, in case you have need write access.?

I'm fine with your proposal, I just need to find out, what team X and Y could be :)

Copy link
Contributor

Choose a reason for hiding this comment

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

Looks fine. Thx

@TBonnin
Copy link
Contributor

TBonnin commented Feb 21, 2017

@sndrs @gtrufitt You don't have to but without it github would not assign commits to you if your local git config uses an email different from your github account :)

@gu-stav gu-stav merged commit 7aecfce into master Feb 21, 2017
@gu-stav gu-stav deleted the docs-enhance-quickstart branch February 21, 2017 17:17
@prout-bot
Copy link
Collaborator

Seen on PROD (merged by @gustavpursche 21 minutes and 47 seconds ago) Please check your changes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants