Skip to content

Commit ea6c8ba

Browse files
authored
Update code_review.yml
1 parent b2724c4 commit ea6c8ba

File tree

1 file changed

+5
-14
lines changed

1 file changed

+5
-14
lines changed

.github/ISSUE_TEMPLATE/code_review.yml

Lines changed: 5 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -12,24 +12,17 @@ body:
1212
description: Check each of the following satisfied functional requirements
1313
options:
1414
- label: "Code solves the issue presented"
15-
required: True
1615
- label: "Code functions, even if not fully solving the issue presented"
17-
required: True
1816
- label: "Performance of the code is acceptable"
19-
required: True
2017
- type: checkboxes
2118
attributes:
2219
label: Code style and legibility
2320
description: Check each of the following checkboxes that describe code legibility
2421
options:
2522
- label: The code contains comments
26-
required: True
2723
- label: Variable names are accurate and appropriate
28-
required: True
2924
- label: The code is easy to read
30-
required: True
3125
- label: The code is formatted correctly
32-
required: True
3326
- type: markdown
3427
attributes:
3528
value: |
@@ -40,25 +33,23 @@ body:
4033
description: Check each of the following checkboxes that describe code legibility
4134
options:
4235
- label: The author understands the arithmetic operators used
43-
required: True
4436
- label: The author described the role of variables with adequate detail
45-
required: True
4637
- label: The author clearly connected the code to the problem presented in the assignment
47-
required: True
4838
- label: The author described their process well, communicating confidence and understanding of their own code
49-
required: True
5039
- type: textarea
5140
attributes:
5241
label: The Good, The Bad, The Ugly
5342
description: Write a short note with at least 3 aspects done well, and 3 aspects on which the code could be improved
5443
value: |
55-
Positives
44+
#### Positives
45+
5646
1.
5747
2.
5848
3.
59-
Improvements
49+
50+
####Improvements
51+
6052
1.
6153
2.
6254
3.
6355
validations:
64-
required: true

0 commit comments

Comments
 (0)