Skip to content

Dice-roll_game - #740

Closed
just-ctrlC-ctrlV wants to merge 0 commit into
thinkswell:masterfrom
just-ctrlC-ctrlV:master
Closed

just-ctrlC-ctrlV wants to merge 0 commit into
thinkswell:masterfrom
just-ctrlC-ctrlV:master

Conversation

@just-ctrlC-ctrlV

Copy link
Copy Markdown
Contributor

Developer Checklist

  • Followed guidelines mentioned in the readme file.
  • Followed directory structure. (e.g. ProjectName/{USERNAME}/...yourfiles)
  • Starred ⭐ the Repo (Optional)

Summary

Dice Duel is an exciting two-player game around rolling dice and competing to achieve the highest score. The game is played with standard six-sided dice, and each player takes turns rolling the dice to accumulate points. The objective is to outscore your opponent by the end of a predetermined number of [rounds.]

Screenshot

Screenshot 2023-10-08 221001

Live Project Link

add a working project link here

@just-ctrlC-ctrlV

Copy link
Copy Markdown
Contributor Author

@PBJI review and merge this request please

@PBJI PBJI left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @just-ctrlC-ctrlV , your repository requires some changes before it can be merged:


❌ RENAMING: You should considering adding your project in the pre-existing Dicegame folder and not Dice_roll_game folder.


❌ DIRECTORY: Your project files, including the README.md should be inside another folder called Username, so ideally your pull request changes would look something like this:

javascript-mini-projects
|           
-------- Dicegame
           |
            ----- just-ctrlC-ctrlV
                   |
                    -------- **Your project files and folders**
                   |
                    --------- README.md (optional yet preferred)

😅 ADDING (Optional): And consider adding/updating README.md with 

  1. Summary about your project
  2. How to setup and use 
  3. Add the screenshots in it.

❓ REFERENCES:

  1. Refer to How to raise the PR for more information

Once you make these necessary changes, please revert back with a comment mentioning @PBJI so that I get notified.

Thank you. 😄

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.

2 participants