-
-
Notifications
You must be signed in to change notification settings - Fork 149
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
Fixing operatingsystem for Amazon Linux #111
Conversation
I was just about to patch this in! Can you please approve the PR? |
@bleiva can you rebase against latest master? This should make travis green |
Merging since this is so trivial. Rebasing is best practices, but people only have so much time. |
Rebasing is tricky business sine it kills many fine features of git's merge abilities. I'm a little scared of it, and all the books and manpages warn against mixing rebase and merge in the same repo, which sounds like very sensible advice to me. merge is the default in git, and in most github projects I have bumped into to date, so I'll stick to that :) |
Hi @seefood, sounds interesting, I have never heard of that also never experienced issues. Do you have a link to that information? |
There's no problem mixing merge and rebase in the same repository. But you shouldn't mix merges and rebases in the same PR/branch divergence. Rebasing a PR onto master is still best practices since it neither scrambles the history, nor any of the "fine features" take away the possibility to reason about your PR. Not saying merging is bad or the features wouldn't be fine in most cases; in case of revitalizing PRs however, rebasing is a very safe strategy. |
@bastelfreak either you were lucky or you are aware of the possible pitfalls of rewriting commit history and manage to steer carefully around them. when rebase rewrites the history, it gives commits new hashes, so diamond merges don't have history to work with and such. So if not careful, you end up patching the same change twice and other headaches that would not happen if the history was not lost. |
Well, I had a few cases where I could not wait for a repo owner to accept a PR and pulled it into my branch, knowing it would be merged by the time I send in my PR. with rebase it would have meant the same changes would be merged with different hashes, which may mean a painful manual merge. Maybe there are better examples, I'm just glad I have not had to deal with them. |
Fixing operatingsystem for Amazon Linux
We had an issue with Amazon Linux servers. This fix is for avoiding the legend 'Operating system is not supported'