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

Adding Information on How to connect Github to Atom #30

Merged
merged 2 commits into from
Jan 13, 2020

Conversation

miguelmartins17
Copy link
Contributor

I Think its important to explain to everyone how they can connect GitHub to Atom and how they can edit Github files from there.

I Think its important to explain to everyone how they can connect GitHub to Atom and how they can edit Github files from there.
@nelsonic nelsonic self-assigned this Jan 8, 2020
@nelsonic nelsonic added the awaiting-review An issue or pull request that needs to be reviewed label Jan 8, 2020
@nelsonic nelsonic added this to To do in Nelson's List via automation Jan 8, 2020
@nelsonic nelsonic moved this from To do to In progress in Nelson's List Jan 8, 2020
Copy link
Member

@nelsonic nelsonic left a comment

Choose a reason for hiding this comment

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

@miguelmartins17 this is a really good addition for people who are new to using GitHub with Atom. 🎉
The only improvements I would make are:

  1. when explaining something for a @dwyl tutorial repository, try to use a dwyl repository in the example.
    You're using https://github.com/miguelmartins17/testrep hen you can easily use https://github.com/dwyl/dev-setup for consistency.
  2. If you can constrain a screenshot to only show the essential information and no extra parts of the screen that do not add any information, do it. The smaller the screenshots the less bandwidth is used downloading them and the less time people waste scrolling.
    Please see: https://github.com/dwyl/dev-setup/blob/af38341a822a0ba7dc693bd5e0313983f088fea8/readme.md#how-to-connect-github-to-atom-step-by-step
    Some of the screenshots are huge when the could be constrained. e.g:
    big
    Could be cut down to a fraction of the size without losing any significance:
    testrep-sample-screenshot

These may seem like minor issues that are not worth "fixing".
But remember that the first item on @dwyl's mission is to "never waste anyone's time".
That means optimising each step to avoid any unnecessary effort (wasted time).
In this case the content you have created is really good. ✅
It can just be "tightened" up to improve the speed that the reader will view it. 👍

@nelsonic nelsonic assigned miguelmartins17 and unassigned nelsonic Jan 8, 2020
@nelsonic nelsonic added in-progress An issue or pull request that is being worked on by the assigned person and removed awaiting-review An issue or pull request that needs to be reviewed labels Jan 8, 2020
@nelsonic nelsonic moved this from In progress to To do in Nelson's List Jan 12, 2020
@miguelmartins17
Copy link
Contributor Author

@nelsonic I'm going to Update this right now please when you have time give me a feedback about this. 👋

Updating This Repository
@nelsonic nelsonic added awaiting-review An issue or pull request that needs to be reviewed and removed in-progress An issue or pull request that is being worked on by the assigned person labels Jan 13, 2020
Copy link
Member

@nelsonic nelsonic left a comment

Choose a reason for hiding this comment

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

@miguelmartins17 looks good. thanks. 👍

@nelsonic nelsonic merged commit 16d3987 into dwyl:master Jan 13, 2020
Nelson's List automation moved this from To do to Done Jan 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-review An issue or pull request that needs to be reviewed
Projects
No open projects
Nelson's List
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants