-
Notifications
You must be signed in to change notification settings - Fork 659
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
Commit 1 rough frame #143
base: main
Are you sure you want to change the base?
Commit 1 rough frame #143
Conversation
index = 0; | ||
while (index < noOfDices) { | ||
var value = value + array[index]; | ||
index += 1; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'index' is not defined.
var numberCoversion = function (array) { | ||
index = 0; | ||
while (index < noOfDices) { | ||
var value = value + array[index]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'index' is not defined.
|
||
var numberCoversion = function (array) { | ||
index = 0; | ||
while (index < noOfDices) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'noOfDices' is not defined.
@@ -1,4 +1,158 @@ | |||
var playerOneRunningCount = 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'playerOneRunningCount' is assigned a value but never used. Allowed unused vars must match /main/u.
@@ -1,4 +1,158 @@ | |||
var playerOneRunningCount = 0; | |||
var playerTwoRunningCount = 0; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'playerTwoRunningCount' is assigned a value but never used. Allowed unused vars must match /main/u.
Code Climate has analyzed commit 4f12b10 and detected 38 issues on this pull request. Here's the issue category breakdown:
Note: there are 32 critical issues. View more on Code Climate. |
Please fill out the survey before submitting the pull request. Thanks!
🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀🚀
How many hours did you spend on this assignment?
Please fill in one error and/or error message you received while working on this assignment.
What part of the assignment did you spend the most time on?
Comfort Level (1-5):
Completeness Level (1-5):
What did you think of this deliverable?
Is there anything in this code that you feel pleased about?