Closed cdavidjean closed 6 years ago
@cdavidjean, thanks for this PR! I'll take a look at it later today. In the meantime, can you update from the latest from the main repo to resolve the conflicts?
I don't know how it happened, but now I have no idea how to solve this?
This branch is 9 commits ahead, 2 commits behind dart-atom:master.
There's no merge option I can find (I'm halfway through reading git for dummies -- but I have used version control software so much better than this...)
On Mon, Aug 7, 2017 at 4:27 PM, Devon Carew notifications@github.com wrote:
@cdavidjean https://github.com/cdavidjean, thanks for this PR! I'll take a look at it later today. In the meantime, can you update from the latest from the main repo to resolve the conflicts?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/dart-atom/dartlang/pull/1137#issuecomment-320678924, or mute the thread https://github.com/notifications/unsubscribe-auth/AdDFww0GvesAQFl8yLHYun-0C45Yhsstks5sVx7fgaJpZM4Ouz7E .
Git can be rough, no doubt about it. It's taken me awhile to get right by git. When using it early on, my resource for many situations was to rm -rf the repo and start over :(
Can you list the remotes you have for this repo (git remote -v
)?
djean@djeanz440:~/github/dartlang$ git remote -v
dart-atom https://github.com/dart-atom/dartlang (fetch) dart-atom https://github.com/dart-atom/dartlang (push) origin git@github.com:cdavidjean/dartlang (fetch) origin git@github.com:cdavidjean/dartlang (push)
Thanks for your help, I'll get it right someday ;)
On Mon, Aug 7, 2017 at 5:13 PM, Devon Carew notifications@github.com wrote:
Git can be rough, no doubt about it. It's taken me awhile to get right by git. When using it early on, my resource for many situations was to rm -rf the repo and start over :(
Can you list the remotes you have for this repo (git remote -v)?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/dart-atom/dartlang/pull/1137#issuecomment-320691820, or mute the thread https://github.com/notifications/unsubscribe-auth/AdDFw1ASMlaGg6bUmL_nE_lroRxGpPmoks5sVymcgaJpZM4Ouz7E .
I create a clean folder and pulled and merge both masters (this and mine) and console.dart ended up, after manual merging, be the same as the one in my master. It seems that accepting my copy of console.dart would be the 'merge' ?
Hmm, not sure why that would be. These steps might help you out:
git clone https://github.com/dart-atom/dartlang
Then, from the github UI, you should have the option of creating a new PR, from cdavidjean:view_management into upstream:master. Any changes on the view_management branch (from review changes) can continue to be pushed to your origin fork via git push origin view_management
. After it's merged, you can git checkout master
and git pull
. That will actually pull from the upstream repo (the main one) because of the initial rename.
Ping me if you have any issues :)