Rework how we explain PRs to students

[Poonam-raj]

Which module(s) and week(s) does this change affect?
Module(s): Onboarding / S+T in ITP

What is the work that needs to be done?

A combination of:

• looking at the existing code review guide and checking that is guides students correctly through how they should engage with a PR.
• Check ITP curriculum to signpost to the above code review guide better - maybe students are missing this guidance completely.

Why is this work important to do?

For Jan ITP we saw a lot of people falling into traps when trying to make their PR pass the GitHub bots. A few changes have already been made to make the PR template even clearer about what we require but the documentation around this might also need amending to follow through on this and signpost clearer.

[Poonam-raj]

• Signpost better to the "Trainee PR Guide" in the first three backlog items that require PR to be done.
• Amend mention of AI tools or reduce importance of it over the PR process itself?
• Embed how to make a PR into the prep material for Onboarding Sprint 1?