fix: precommit lint and format

[Bushuo]

Ref: #98

These changes seem to work on my end now.
I also added two more npm scripts to be invoked from the CI job.

Could someone please verify it?

Basics

• Details of what you changed are in commit messages
  (first line should have module: short statement syntax)
• References to issues, e.g. close #X, are in the commit messages.
• The buildserver is happy.
• The PR is rebased with current master.

Checklist

• I added unit tests for my code
• I fully described what my PR does in the documentation
  (not in the PR description)
• I fixed all affected documentation
• I fixed all affected decisions
• I added code comments, logging, and assertions as appropriate
• I mentioned every code not directly written by me in reuse syntax

Review

• Documentation is conforming to our Documentation Guidelines
• Examples are well chosen and understandable
• Code is conforming to our Coding Guidelines
• Code is consistent to our Design Decisions

[buenaflor]

lgtm, I'll try it out today as well

[Benjaminimal]

Looks good.
I've got a few notes though.

1. I checked out this branch, rebased it on master, and ran npm run lint and got this warning. Warning: React version not specified in eslint-plugin-react settings. See https://github.com/jsx-eslint/eslint-plugin-react#configuration .
   You can silence this if you add the following to .eslintrc.json.

  "settings": {
    "react": {
      "version": "detect"
    }
  }

2. Could you take a look at the lining errors and maybe fix them? Else we will have a check in the CI soon that always fails.
3. Please run npm run format, commit it, and add the commit hash with a brief comment to .git-blame-ignore-revs file.
   https://www.stefanjudis.com/today-i-learned/how-to-exclude-commits-from-git-blame/#ignore-commits-via-%60--ignore-revs-file%60

For 2. and 3. please rebase this branch on master.

[Benjaminimal]

I know you only split this up but did not write it.
I have heard that people turn this hook off because building the front end on each commit takes too much time.
It would make more sense to get rid of the build part, only run formatting and lining and let the CI make sure the project can build.

[Benjaminimal]

Nevermind, I took care of it with this PR.
#159

[Benjaminimal]

Forget point 3.

Reformatting, however, should be done after Milestone 1 (#35), to not postpone the Milestone.

Seems like Markus wants to wait with it.

[Bushuo]

@Benjaminimal great work :)
I will close this then

[markus2330]

It can stay open, hopefully we are soon done with the milestone and then we can merge it.

[markus2330]

Ahh, now I see, it got obsolete because of #160? So I'll close it again.