Skip to content

Fast alphametics solver (uses simplified partial equations to limit…#476

Closed
moveson wants to merge 1 commit intoexercism:masterfrom
moveson:alphametics-fast-solver
Closed

Fast alphametics solver (uses simplified partial equations to limit…#476
moveson wants to merge 1 commit intoexercism:masterfrom
moveson:alphametics-fast-solver

Conversation

@moveson
Copy link
Copy Markdown
Contributor

@moveson moveson commented Oct 31, 2016

…set of viable permutations)
@Insti Please review and merge with PR #469.

@Insti
Copy link
Copy Markdown
Contributor

Insti commented Oct 31, 2016

Awesome, thanks @moveson ❤️

@Insti
Copy link
Copy Markdown
Contributor

Insti commented Oct 31, 2016

Why does it think there are conflicts?
(Not that it really matters, but I'm guessing if you rebased this branch off master it would fix it.)

@moveson moveson closed this Oct 31, 2016
@moveson moveson deleted the alphametics-fast-solver branch October 31, 2016 17:42
@kotp
Copy link
Copy Markdown
Member

kotp commented Nov 1, 2016

Why was this closed?

@moveson
Copy link
Copy Markdown
Contributor Author

moveson commented Nov 1, 2016

My branch was behind and that was creating conflicts. I deleted the branch, created a new one, and opened a new PR #477.

@kotp
Copy link
Copy Markdown
Member

kotp commented Nov 1, 2016

Hopefully next time you have it figured out so that won't need to happen. You can always bring your branch up to date. :) I am happy to work through git issues like this with you, well worth knowing how to do that.

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.

4 participants