Skip to content

Commit f00380d

Browse files
committed
ci: review tweak
1 parent c00d488 commit f00380d

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

.github/workflows/review.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,6 +74,7 @@ jobs:
7474
</pr-description>
7575
7676
Please check all the code changes in this pull request against the style guide, also look for any bugs if they exist. Diffs are important but make sure you read the entire file to get proper context. Make it clear the suggestions are merely suggestions and the human can decide what to do
77+
When critiquing code against the style guide, be sure that the code is ACTUALLY in violation, don't complain about else statements if they already use early returns there. You may complain about excessive nesting though, regardless of else statement usage.
7778
7879
Use the gh cli to create comments on the files for the violations. Try to leave the comment on the exact line number. If you have a suggested fix include it in a suggestion code block.
7980

0 commit comments

Comments
 (0)