Update Checklist in PR template

[joergbrech]

I updated the checklist in the PR template

Description

We want to be more disciplined in actually checking the template. In many PRs, the tasks in the checklist are meaningless, which means in many PRs they are never checked and simply ignored. I think this causes us as contributors and reviewers to be a bit lax. I updated it so that both contributors and reviewers must add a check in each row.

Checklist:

• A test for the new functionality was added. Does not apply
• All tests run without failure. Does not apply
• The new code complies with the TiGL style guide. Does not apply
• New classes have been added to the Python interface. Does not apply
• API changes were documented properly in tigl.h. Does not apply
• Changes were documented in tigl/ChangeLog.md Does not apply

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.47%. Comparing base (8c80c8b) to head (2aa4991).
Report is 3 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1038   +/-   ##
=======================================
  Coverage   69.47%   69.47%           
=======================================
  Files         301      301           
  Lines       26889    26889           
=======================================
  Hits        18681    18681           
  Misses       8208     8208           

Flag	Coverage Δ
unittests	69.47% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

[svengoldberg]

I like the idea! Out of my own experience, I think the old version might need an update 👍

[svengoldberg]

To me, everything looks fine 👍