Sitelet https://web.archive.org/web/20201101052225/https://github.com/github/docs/pull/587
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

added Git Handbook link to further reading section #587

Merged
merged 4 commits into from Oct 23, 2020

Conversation

@krishnadevz
Copy link
Contributor

@krishnadevz krishnadevz commented Oct 16, 2020 •

added Git Handbook link to further reading section

Why: In the further reading section I added one resource that covers every command of GitHub and git with a description that best fit there and that can help to the whole GitHub beginners community.

What's being changed: content/github/using-git/pushing-commits-to-a-remote-repository.md

image

Check off the following:

added Git Handbook link to further reading section
@deploy-to-heroku deploy-to-heroku bot requested a deployment to docs-587--patch-2 Oct 16, 2020 Pending
@krishnadevz
Copy link
Contributor Author

@krishnadevz krishnadevz commented Oct 19, 2020 •

can you guys review my pull request?

@github-actions github-actions bot added this to Triage in Docs team reviews Oct 19, 2020
@janiceilene
Copy link
Contributor

@janiceilene janiceilene commented Oct 20, 2020

@krishnadevz Thanks for opening a PR 👍 Please fill out the pull request template in the top comment in this pull request, then we'll get this up for review!

In the future, please do not @ mention any other person in an issue or PR, unless they are already participating in that conversation.

@krishnadevz
Copy link
Contributor Author

@krishnadevz krishnadevz commented Oct 21, 2020 •

@krishnadevz Thanks for opening a PR 👍 Please fill out the pull request template in the top comment in this pull request, then we'll get this up for review!

In the future, please do not @ mention any other person in an issue or PR, unless they are already participating in that conversation.

okay got it I will not @ in the future but I have done changes

@deploy-to-heroku deploy-to-heroku bot temporarily deployed to docs-587--patch-2 Oct 21, 2020 Inactive
Copy link
Contributor

@shati-patel shati-patel left a comment

Thank you for the change! I'll get this merged shortly 😃

@deploy-to-heroku deploy-to-heroku bot temporarily deployed to docs-587--patch-2 Oct 23, 2020 Inactive
@shati-patel shati-patel merged commit 233ee4e into github:main Oct 23, 2020
12 checks passed
12 checks passed
triage
Details
automerge
Details
run run
Details
see_if_should_skip
Details
lint
Details
test (content)
Details
test (meta)
Details
test (rendering)
Details
test (routing)
Details
test (unit)
Details
test (links-and-images)
Details
staging deployed in 75 seconds
Details
Docs team reviews automation moved this from Triage to Done Oct 23, 2020
@github-actions
Copy link

@github-actions github-actions bot commented Oct 23, 2020

Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours.

@krishnadevz krishnadevz deleted the krishnadevz:patch-2 branch Oct 23, 2020
@github github deleted a comment Oct 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.