carpentries-incubator / carpentries-incubator/swc_github_flow

Just confirmed: Pulling from this repo to mine, puts my repo 1 ahead, not even

Open
#20 2 comments 0 reactions 0 assignees View on GitHub
bug help wanted
Dominant language
No language data
Stars
12
Forks
9
PR merge metrics
No merged PRs in 30d

Description

First this is a GREAT addition to the Git introduction for beginners, maybe the best I've seen. It carefully worded, the images are detailed, it has a glossary, it's really great.

But here's the issue. If I follow the instructions **exactly** using the GUIs, my repo at https://github.com/hoytpr/swc_github_flow, (in this case using the "master" branch, but it's the same on gh-pages) always ends up as "one commit ahead of dmgt:master." Example: When I checked this morning, my master branch was three commits behind dmgt:master. Following the instructions, I set up the merge request, switching the base, so that I was creating a pull request from dmgt/swc_github_flow master branch to hoytpr/swc_github_flow master branch. Everything was fine and able to merge, so I merged the 3 commits into my repo.

After confirming the merge, `for_novice_contributors.md` says (on line 111) "The status should now be “Merged”, and your repository will be up to date with all the changes from the remote repository!"

But my repository says "This branch is 1 commit ahead of dmgt:master. ".
![Novice-flow-after-merge](https://user-images.githubusercontent.com/5910884/75047654-c0802680-548c-11ea-8c28-10248e64fcee.png)
This always happens and is remarkably difficult to fix for a novice, I don't think it can be fixed from the GUI.

I know this isn't a big deal *now*, but when I was a novice this prevented me from making changes and putting in a PR to the Carpentries... **because I was sure I'd done something wrong, and didn't even know what was in the the extra commit.** If you look at the PR within the GUI, there is nothing indicating what is contained in the extra commit (sorry another big image attached):
![Novice-flow-after-merge-compared](https://user-images.githubusercontent.com/5910884/75048013-751a4800-548d-11ea-97a1-5f8f18bb274a.png)

Maybe we should simply state "The status should now be “Merged”, and your repository will be up to date with all the changes from the remote repository, and one commit ahead (the pull request you just made) compared to the remote repository! You can ignore the extra commit, and start making changes on a new branch". Note this means when you make changes on a test branch and it then says you are "two commits" ahead of the remote repo, not "a commit" as stated on line 125.

This is such a great introduction, and the difference is probably trivial to most folks, it's just that it wasn't trivial when it happened to me. The fix has to go through your LOCAL repo (your desktop or laptop) where you `fetch` the upstream repo (e.g. dmgt/swc_github_flow master) and `merge` it locally, then `push` it to your remote GitHub repo (e.g. hoytpr/swc_github_flow master) without a commit.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.