Skip to content

Merge all the review instructions in one file - #106

Open
LonMcGregor wants to merge 1 commit into
mainfrom
lm-update-review-guide
Open

Merge all the review instructions in one file#106
LonMcGregor wants to merge 1 commit into
mainfrom
lm-update-review-guide

Conversation

@LonMcGregor

Copy link
Copy Markdown
Contributor

Now that we are reviewing this project, make the instructions and criteria easier for reviewers to find

@LonMcGregor LonMcGregor added the NotCoursework A PR that edits material in the module. Needs review by the curriculum team. label Sep 2, 2026
Comment thread HOW_TO_REVIEW.md
-->

# Everyone reviews code at CYF
# TV Show Project Requirements

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nitpick: ideal would be to only have one h1 per page, so perhaps this could be structured with a single top-level heading and then change the others to h2s

# How To Review

## TV Show Project Requirements

...

## Reviewing the paired project

...

## General Advice for Reviewers

...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NotCoursework A PR that edits material in the module. Needs review by the curriculum team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants