Improving contributing guide

[taliaga]

Updating contributing guide:

• Change order in "Getting started" section to promote self-review before push
• Clarify Review Process section, confusing use of owner term?
• Images: screenshots vs diagrams, promoting diagrams as code :)

Fun facts worth noting:

• The contributing guide for updating the contributing guide is the same contributing guide
• Some contributions to the contributing guide could eventually be both a violation and an improvement of the guide

[github-actions]

preview available: https://docs.tds.cscs.ch/194

[github-actions]

preview available: https://docs.tds.cscs.ch/194

[github-actions]

preview available: https://docs.tds.cscs.ch/194

[msimberg]

Thanks @taliaga! I'll refer you to this comment for now about the spell checker: #177 (comment).

Edit: buttons are hard. Fixed the link.

[msimberg]

A few non-blocking comments. This looks like a nice addition to me. Thanks @taliaga!

[msimberg]

I think screenshot could be added to .github/actions/spelling/allow.txt. Wikipedia seems to use "screenshot" as well (not "screen shot"), so the spell checker is wrong here.

[taliaga]

adding codeowners and screenshots. I wonder which tool/config are you using to keep the alphabetical order? my sort yields something different so I just placed them manually

[msimberg]

Ah, fair question. Emacs' sort-lines 🙃 I suppose mine sorts caps before lower case, whereas yours maybe mixes them?

[github-actions]

preview available: https://docs.tds.cscs.ch/194

[github-actions]

preview available: https://docs.tds.cscs.ch/194

[taliaga]

Thanks reviewers