Skip to content

Add OCR Numbers Test Generator #449

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

Closed
wants to merge 11 commits into from
Closed

Conversation

GKotfis
Copy link
Contributor

@GKotfis GKotfis commented Oct 2, 2017

This reference #417 issue.

@GKotfis GKotfis closed this Oct 2, 2017
@ErikSchierboom
Copy link
Member

@GKotfis May I ask why did you close this PR?

@GKotfis
Copy link
Contributor Author

GKotfis commented Oct 2, 2017

I don't know why older commits was added to pull request. Should be only d30d974, a6419ba, 3b049fb, a7e0c6c

@ErikSchierboom
Copy link
Member

@GKotfis Ah, okay. You should be able to fix that by selective dropping commits in an interactive rebase. So you do:

git fetch exercism
git rebase -i exercism/master

And then in the interactive rebase window, you can drop commits. Perhaps make a copy branch first? :)

@robkeim
Copy link
Contributor

robkeim commented Oct 2, 2017

+1 for making a copy first... my first rebase went wrong and then I got to spend a long time figuring out a bunch of low level git commands and black magic to try to revert my changes :)

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