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

Markdown #933

Merged
merged 4 commits into from
Mar 2, 2016
Merged

Markdown #933

merged 4 commits into from
Mar 2, 2016

Conversation

magsout
Copy link
Member

@magsout magsout commented Mar 2, 2016

I added new Component Markdown to handle element from Github without ClassName.

r? @miketaylr

@magsout
Copy link
Member Author

magsout commented Mar 2, 2016

Oops, neeed to rebase

@magsout
Copy link
Member Author

magsout commented Mar 2, 2016

@miketaylr I need you. I'm not an expert of git. When I try to fixed conflict after rebased I'm lost...

@miketaylr
Copy link
Member

@magsout np! let me try to fix it up and push.

@miketaylr
Copy link
Member

OK, should be good to review now. 👍

@miketaylr
Copy link
Member

Hm, weird error:

Running "cssnext:dist" (cssnext) task

Warning: /home/travis/build/webcompat/webcompat.com/webcompat/static/css/development/main.css:54:1: Failed to find 'components/Markdown' from /home/travis/build/webcompat/webcompat.com

cssnext:dist task works for me locally, maybe I need to kick the Travis cache.

@@ -51,3 +51,4 @@
@import "components/label";
@import "components/upload-form";
@import "components/contributors";
@import "components/Markdown";

This comment was marked as abuse.

@magsout
Copy link
Member Author

magsout commented Mar 2, 2016

Ah yes, I usually write Markdown instead of markdown on my other project. But in this case @import component/markdown is good

@miketaylr
Copy link
Member

LGTM. :shipit:

miketaylr pushed a commit that referenced this pull request Mar 2, 2016
@miketaylr miketaylr merged commit f6ef17f into master Mar 2, 2016
@miketaylr miketaylr deleted the markdown branch March 2, 2016 23:23
@magsout
Copy link
Member Author

magsout commented Mar 3, 2016

@miketaylr thanks, but curious, how did you do ?

@miketaylr
Copy link
Member

Sure!

So to rebase against master from a branch (not a fork) I usually do something like this:

git fetch
git pull --rebase origin master

And then when there's a conflict I do git status to see which files were "both modified" (or whatever the message is).

I open those in my editor and search for <<<< to jump to the spot where the conflict is. This is the tricky part -- you have to decide which parts should stay and which parts go. This can be dead simple, or it can be confusing if patches change things around a lot.

https://help.github.com/articles/resolving-a-merge-conflict-from-the-command-line/#creating-an-edit-collision should help here a little. But it can still be a pain in the butt to get right.

Once you think you've got it right, you have to git add file-that-you-just-edited to let git know that you're done with the conflict resolution. And after that, git rebase --continue.

You might get another conflict from the next patch it applies in the rebase. So rinse and repeat until it tells you you're done.

@magsout
Copy link
Member Author

magsout commented Mar 3, 2016

Once you think you've got it right, you have to git add file-that-you-just-edited to let git know that you're done with the conflict resolution. And after that, git rebase --continue.

OK, what I missed when I tried to rebase... Thanks for the explanation, 👍

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

Successfully merging this pull request may close these issues.

2 participants