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

Use grep -E in lint.sh to make it compatible with BSD grep #67

Merged
merged 4 commits into from
Dec 3, 2016
Merged

Use grep -E in lint.sh to make it compatible with BSD grep #67

merged 4 commits into from
Dec 3, 2016

Conversation

maksimov
Copy link
Contributor

@maksimov maksimov commented Dec 3, 2016

No description provided.

@zimmski
Copy link
Collaborator

zimmski commented Dec 3, 2016

Please squash bf5b7ce into the previous commit to keep the history clean.

@maksimov
Copy link
Contributor Author

maksimov commented Dec 3, 2016

I'm not sure if this is better, since I now have a merge commit there too. Shall I just nuke it and do it again? AFAIK, you can also squash while merging on Github.

@zimmski
Copy link
Collaborator

zimmski commented Dec 3, 2016

I am not really sure how the squash feature behaves on GitHub. I will take a look. For example how do they handle signed commits?

For this PR I meant the squash functionality of git using rebase. Maybe you can have a look at https://git-scm.com/book/en/v2/Git-Tools-Rewriting-History#Squashing-Commits Fixups (can be also don in the interactive rebase mode) is also a great way of squashing commits http://fle.github.io/git-tip-keep-your-branch-clean-with-fixup-and-autosquash.html

@maksimov
Copy link
Contributor Author

maksimov commented Dec 3, 2016

That's how I did it, but then I had to rebase with my fork so this is how there's a merge commit.

@zimmski zimmski merged commit 7eb553d into sergi:master Dec 3, 2016
@zimmski
Copy link
Collaborator

zimmski commented Dec 3, 2016

I tried out the squash feature. To be honest, I rather prefer to do merges in the future. The information on where the commit came from is lost.

If you have a merge commit a merge was somehow issues, e.g., via pull. Did you try "git checkout --all --prune && git checkout yourbranch && git rebase yourforksremotename/master"? That usually does it for me.

Anyway, thanks for the PR! I really appreciate it :-)

@maksimov
Copy link
Contributor Author

maksimov commented Dec 3, 2016

Wow, never did anything like that. My upstream didn't mind an extra commit. Btw, I notice that merge commit didn't really show up on Github, and you can't really get away without them where more than one person is making changes to the upstream. Anyway, I'll try your suggestion next time. Thanks!

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