-
Notifications
You must be signed in to change notification settings - Fork 126
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
Dice Game #31
Dice Game #31
Conversation
Hello. Thanks for checking out the repo and contributing. :) I will review this pull request as soon as I have time to do so. |
I made some changes:
And I made some additions:
|
Thank you so much! This was my first time working on something from GitHub, thank you for assisting me!
Best Regards,
Javier
… On Apr 12, 2022, at 5:49 PM, Zachary Patten ***@***.***> wrote:
I made some changes:
f0801a3 moved the project into the Projects directory because that is where all the projects are
f1ada2d I changed Console.ReadKey() to Console.ReadKey(true) because this hides the user input (for example if the user types the A key)
f3b01e8 added a description of the game that shows when the application is launched
ff86f29
changed the turn order so that the Rival rolls the dice first as I felt it would add more anticipation because the player knows what they need to roll to win.
removed the Thread.Sleep because I don't think it is entirely necessary here
updated the output verbiage a bit
638994a removed class Program so that the code is top-level statements because it is shorter
4a168b7 used string interpolation to simplify the code from
"The score is now - You : " + playerPoints + ". Rival : " + rivalPoints + "." to
$"The score is now - You : {playerPoints}. Rival : {rivalPoints}."
cd9290b moved the declaration of rivalRandomNum and playerRandomNum inside the loop rather than before because that is generally the preferred place to declare variables
And I made some additions:
43d9c5f added a blazor port of the game so that it will appear on the website and be playable online :)
7acfdff added the necessary files so that Dice Game shows up as an option when using Visual Studio Code
eb2dfac added a GitHub Actions workflow for build validation
c18112d added Dice Game to README.md so it shows up on the home page of the GitHub repo and and added a Projects\Dice Game\README.md
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.
|
This pull request has bene merged. Thanks @javithegreat35 for contributing a new game, and congratulations on your first pull request on GitHub! If you want to show off your game, it is playable at this link: https://zacharypatten.github.io/dotnet-console-games/Dice%20Game |
By the way, yours was the first game to be contributed and not originally written by myself (other contributors contributed to existing games). You already show up in the contributors tab of the repo, but if you would to be explicitly mentioned on that game let me know. For example...
...could be added in the |
Hello,
Sorry for the late response. Yes, you could mention me anywhere you see fit! Again thank you for all your help!
…Sent from my iPhone
On Apr 19, 2022, at 12:23 PM, Zachary Patten ***@***.***> wrote:
By the way, yours was the first game to be contributed (other contributors contributed to existing games). If you would like that to be mentioned anywhere let me know.
For example,
contributors: @javithegreat35
in the Notes column of the table in the README.md.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.
|
merge Dice Game
Hello, I have created a new game called "Dice Game" which has the player go up against a computer and they will each roll a dice 10 times and the player who gets the higher number 10 times wins the game.