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

Resolve Issue #809 #812

Closed
wants to merge 4 commits into from

Conversation

AnthonyMichaelTDM
Copy link
Contributor

#809 brought to my attention that I had mistakenly pushed the same code to two games (pushed 91_train to 91_train and 92_trap), this PR removes the rust implementation of 92_trap (which was really just 91_train), I'll probably contribute an actual implementation of 92_trap soon.

Sorry about this, but glad it's fixed now :)

@ugurkupeli
Copy link
Contributor

Just read your comment about doing 92 Trap soon after doing it myself, hope you don't mind :)

@AnthonyMichaelTDM
Copy link
Contributor Author

No worries, I was gonna do it later today so I haven't started yet, can you @ mention me in the pr so I can update #595 when you get it done?

@coding-horror
Copy link
Owner

Oops, I created conflicts. My bad!

@AnthonyMichaelTDM
Copy link
Contributor Author

no worries, given that the PR that created the conflict also fixes this issue, I'd say it's safe to just close this PR

@ugurkupeli
Copy link
Contributor

No worries, I was gonna do it later today so I haven't started yet, can you @ mention me in the pr so I can update #595 when you get it done?

Sure thing, I will do that in the future. The 92 Trap PR is complete

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