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

Fix small stuffs #4274

Closed
wants to merge 4 commits into from
Closed

Fix small stuffs #4274

wants to merge 4 commits into from

Conversation

Sija
Copy link
Contributor

@Sija Sija commented Apr 11, 2017

This PR introduces couple of small, harmless tweaks/fixes as the title suggests.

@Sija
Copy link
Contributor Author

Sija commented Apr 11, 2017

Damn, I'm still capable of borking a PR, opening next one...

@Sija Sija closed this Apr 11, 2017
@RX14
Copy link
Contributor

RX14 commented Apr 11, 2017

Why not update this PR with force push?

@Sija
Copy link
Contributor Author

Sija commented Apr 11, 2017

@RX14 I did. After that, commit 6026ccc unintentionally became part of the PR and I didn't know how to get rid of it.

@Sija Sija deleted the fix-small-stuff branch April 11, 2017 20:22
@RX14
Copy link
Contributor

RX14 commented Apr 11, 2017

You should be able to rebase on master and then force push the results. If all else fails, you can always cherry pick your commits to a new branch and force push that to this PR instead.

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.

None yet

3 participants