Skip to content

Feedback#2

Open
CheezItMan wants to merge 1 commit intohalahaddad1:masterfrom
CheezItMan:patch-2
Open

Feedback#2
CheezItMan wants to merge 1 commit intohalahaddad1:masterfrom
CheezItMan:patch-2

Conversation

@CheezItMan
Copy link

JSL Day 3

Requirement Comments
Well formatted code You are using variable names without meaning like c and n etc. You also have a lot of indentation issues.
Number guessing game 👍
Duck-duck-goose game 👍
Flower-picking-game 👍
Double-entry modification 👍

Summary

Your code works, and you hit the main learning goals here, but I would like you to focus on improving your code formatting. As is, it's pretty hard to read.

# JSL Day 3

| Requirement | Comments
|---|---
| Well formatted code |  You are using variable names without meaning like `c` and `n` etc.  You also have a lot of indentation issues.
| Number guessing game | 👍 
| Duck-duck-goose game | 👍 
| Flower-picking-game | 👍 
| Double-entry modification | 👍 

## Summary

Your code works, and you hit the main learning goals here, but I would like you to focus on improving your code formatting.  As is, it's pretty hard to read.
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.

1 participant