Skip to content

done#2

Open
azizomarck wants to merge 2 commits into
hackyourfuturecanada:masterfrom
azizomarck:Aziz-Omar
Open

done#2
azizomarck wants to merge 2 commits into
hackyourfuturecanada:masterfrom
azizomarck:Aziz-Omar

Conversation

@azizomarck
Copy link
Copy Markdown

No description provided.

@epq
Copy link
Copy Markdown
Collaborator

epq commented Sep 12, 2019

@esyasar Please leave some feedback for Aziz!

Comment thread game.js Outdated
// Write all your code here
// Replace the contents of this file with your own code
console.log('Hello, world!');
const choices = ["ROCK","SCISSOR","PAPER"];
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

In the original game, choices are rock, paper, and scissors, but you used scissor it can be confusing for players.

@esyasar
Copy link
Copy Markdown

esyasar commented Sep 15, 2019

I liked your code. It's apparent and, readable. Good job!

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.

4 participants