Skip to content

Adding Comp Questions section to feedback template #47

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.

Sign up for GitHub

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Adding Comp Questions section to feedback template #47

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
4086af7
Adding Comp Questions section to feedback template
CheezItMan Aug 11, 2017
c3d0f34
Update PULL_REQUEST_TEMPLATE.md
CheezItMan Aug 11, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 9 additions & 8 deletions .github/PULL_REQUEST_TEMPLATE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,12 @@
# Random Menu
Congratulations! You're submitting your assignment.
## What We're Looking For

## Comprehension Questions

| Question | Answer |
| Feature | Feedback |
|--- |--- |
| How did you store menu components? Why? | |
| Could you have stored components using a different data structure? For example, if you used an Array could you have used a Hash? | |
| Did you find yourself repeating the same code? Why did you have to do it that way? | |
| What type of loop did you use? Why did you choose that type? | |
| Random Menu of 10 items displayed in the terminal. | |
| Generator pulls one random item from each array to create menu items. | |
| Comprehension questions answered| |
| **Baseline** | |
| Readable code with consistent indentation. | |
| **Extras** | |
| | |