- Notifications
You must be signed in to change notification settings - Fork163
Dice Game#31
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Dice Game#31
Uh oh!
There was an error while loading.Please reload this page.
Conversation
ZacharyPatten commentedApr 12, 2022
Hello. Thanks for checking out the repo and contributing. :) I will review this pull request as soon as I have time to do so. |
ZacharyPatten commentedApr 13, 2022
I made some changes:
And I made some additions:
|
javithegreat35 commentedApr 13, 2022 via email
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 aref1ada2d 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 launchedff86f29 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 bit638994a removed class Program so that the code is top-level statements because it is shorter4a168b7 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 Codeeb2dfac added a GitHub Actions workflow for build validationc18112d 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. |
ZacharyPatten commentedApr 13, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
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 |
ZacharyPatten commentedApr 19, 2022 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
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 |
javithegreat35 commentedApr 25, 2022 via email
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.