MarcusZagorski / Module-HTML-CSS

The central repo for the HTML CSS module
https://cyf-module-html-css.netlify.app/wireframe/
Creative Commons Attribution 4.0 International
0 stars 0 forks source link

[TECH ED] Module Project: Code Review #30

Open MarcusZagorski opened 1 year ago

MarcusZagorski commented 1 year ago

From Module-HTML-CSS created by SallyMcGrath: CodeYourFuture/Module-HTML-CSS#18

Link to the coursework

https://github.com/CodeYourFuture/HTML-CSS-Module-Project/tree/master/level-3

Why are we doing this?

Learning to read and review other people's code is a key skill for a developer. When you join a new team, you will usually also join an existing codebase, and you will need to be able to read it. If you've never looked at code written by other people, you will find this really challenging (it's already hard!).

  1. Do the Github Skills course in reviewing pull requests
  2. Ask for a code review from a colleague on your Karma project.
  3. Complete a code review for a colleague on their Karma project.

Maximum time in hours

3

How to submit

Link the PR you reviewed on your copy of this ticket. Paste the url in comment on your ticket.

MarcusZagorski commented 1 year ago

https://github.com/CodeYourFuture/HTML-CSS-Module-Project/pull/622#pullrequestreview-1657069583