Share code review guidelines

[OleksiyRudenko]

Both this repo and frontend-2019-homeworks need Code Review Guidelines (hereinafter the "CRG") while the latter are part of this repo only.

Possible options to resolve:

• in frontend-2019-homeworks add link to frontend-2019-p2p/code-review-guidelines.md
• move code-review-guidelines.md to kottans/frontend and add links in frontend-2019-* repos
• (worst) make a clone of code-review-guidelines.md and post to frontend-2019-homeworks

Things to consider:

• CRG might be useful for other courses beyond frontend (while code samples are definitely frontend specific)
• ongoing offline course can survive without its own copy of CRG, as we expect students become CR gurus after 2-3 code review sessions

[o-msh]

I can do it according to second options (move .md to kottans/frontend and add links). Also we can append some code review instructions in future. We have guidelines according to FE course except couple of common links in top of instructions. IMO while we doesn't have any different course yet. YAGNI. @OleksiyRudenko what do you think about it?

[zonzujiro]

kottans/frontend is a good option, as for me.

[OleksiyRudenko]

Looks like migration to kottans/frontend is the option. Let's see how things go with p2p model (will it survive, shall we be able to switch to p2p model completely) and if offline course students really need CRG when it is time to conduct peer CR within offline course. No hurry at the time being so we still can wait and see. Meanwhile other opinions may arrive here.

@o-msh , @zonzujiro , thank you for your input, guys!

[OleksiyRudenko]

As we have multiple courses and current course as of time of writing this is backend we may want the CRG be a part of something more global. Luckily we now have kottans/library

[OleksiyRudenko]

Keep #11 in mind if the CRG would migrate somewhere else.