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

Checkers #40

Merged
merged 68 commits into from Jun 23, 2022
Merged

Checkers #40

merged 68 commits into from Jun 23, 2022

Conversation

Wycott
Copy link
Contributor

@Wycott Wycott commented Jun 16, 2022

No description provided.

@ZacharyPatten
Copy link
Collaborator

Thanks for the interest in contributing a #17 new game! :)

I will review this pull request as soon as I have time to do so. Also, I can add the additional files mentioned in checklist on #17.

@ZacharyPatten ZacharyPatten changed the title Initial cut Checkers Jun 16, 2022
@ZacharyPatten ZacharyPatten mentioned this pull request Jun 16, 2022
@ZacharyPatten ZacharyPatten added the community contribution Games originally contributed by members of the community. Thank you! label Jun 17, 2022
@ZacharyPatten ZacharyPatten self-requested a review June 19, 2022 02:07
@ZacharyPatten
Copy link
Collaborator

ZacharyPatten commented Jun 23, 2022

I am happy with the state of this pull request and I am ready to merge it. Since I made numerous changes, @Wycott would you like to review it before I merge it in? I tried to be reasonably granular with the commits and have informative commit messages. I'm hoping all my changes make sense and you agree with them, but if you don't, I want to hear your opinions.

I didn't do a lot of testing, and I'm sure I overlooked a bug or two (probably ones I introduced), but the code can always be modified in a future pull request.

And I wanted to thank you again for the great contribution! ❤️

Projects/Checkers/Board.cs Outdated Show resolved Hide resolved
Projects/Checkers/Game.cs Outdated Show resolved Hide resolved
Projects/Checkers/Game.cs Outdated Show resolved Hide resolved
Projects/Checkers/Player.cs Outdated Show resolved Hide resolved
Projects/Checkers/Board.cs Outdated Show resolved Hide resolved
Projects/Checkers/Board.cs Outdated Show resolved Hide resolved
Projects/Checkers/Game.cs Outdated Show resolved Hide resolved
Projects/Checkers/Program.cs Outdated Show resolved Hide resolved
Projects/Checkers/Program.cs Outdated Show resolved Hide resolved
Projects/Checkers/Program.cs Outdated Show resolved Hide resolved
Projects/Checkers/Program.cs Outdated Show resolved Hide resolved
Projects/Checkers/Program.cs Outdated Show resolved Hide resolved
@ZacharyPatten ZacharyPatten merged commit 39977b7 into dotnet:main Jun 23, 2022
Fuinny pushed a commit to Fuinny/dotnet-console-games that referenced this pull request Jun 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community contribution Games originally contributed by members of the community. Thank you!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants