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

Repository with temporary issues #18

Closed
ghost opened this issue Jan 11, 2013 · 2 comments
Closed

Repository with temporary issues #18

ghost opened this issue Jan 11, 2013 · 2 comments

Comments

@ghost
Copy link

ghost commented Jan 11, 2013

Hello everyone,

I've accepted pull request #17, but it added quite a lot of weight to the repository (it's now around 20 MB). I was able to revert the changes, but the old files remain in the history. I don't know how to erase the commits AND their files.

I asked a question on StackOverflow, and I'm still trying to solve this issue:
http://stackoverflow.com/questions/14264719/how-to-completely-remove-a-merged-pull-request

For now, I will not accept any pull requests to avoid increasing the mess. And please, try not to base new pull requests on this code. I'm trying to solve this ASAP. Sorry for the inconvenience...

For users, feel free to clone and use RouteFlow. It doesn't affect the functionality.

If anyone has any tips, they're more than welcome :)

@joestringer
Copy link
Contributor

Hi Allan,

You can do this by force-pushing the older version. This is generally a bad idea, but I think in this case it is excusable if not many have been using this latest master.

Steps:-

  • Check out the older version (I think you want version 70b29ba ?)
  • git push --force origin HEAD:master
    (where 'origin' is the name of this remote repository)

This will clean up the master repository. In future it is probably best to only merge locally and push to GitHub when you are sure you want to share the changes.

@ghost
Copy link
Author

ghost commented Jan 11, 2013

Thank you very much Joe! It worked perfectly.

I know this would normally be a serious offense. I used GitHub's automerge on impulse, and then it was too late...

@ghost ghost closed this as completed Jan 11, 2013
KitL pushed a commit to KitL/RouteFlow that referenced this issue Dec 16, 2014
Added the OSPF constant for enabling rftest2 in vandervecken branch
This issue was closed.
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

No branches or pull requests

1 participant