Skip to content

Conversation

@jbwhit
Copy link
Contributor

@jbwhit jbwhit commented Mar 25, 2014

After random failures during build, and my failing to rebase properly,
I have re-committed the code that was used in this previous pull
request: #2103

After random failures during build, and my failing to rebase properly,
I have re-committed the code that was used in this previous pull
request: astropy#2103
@jbwhit
Copy link
Contributor Author

jbwhit commented Mar 25, 2014

If a person could restart the build process, I think the build process will pass because only one test failed to run (all tests that ran, passed) because of some git glitch.

@astrofrog
Copy link
Member

@jbwhit - I think it would be best to change the commit message to actually describe what is fixed/implemented and not mention the rebasing problems and the other pull request. You should be able to do this with git amend: http://nathanhoad.net/git-amend-your-last-commit

@embray
Copy link
Member

embray commented Mar 25, 2014

In general creating a new PR is also unnecessary in general--even if you rebase you can push your new commits to the existing branch over the old commits using --force and that will update the existing PR. I will close this one in favor of fixing the existing one.

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.

3 participants