Enforce strict code style to prevent merge conflicts

[jennydaman]

Currently, ChRIS_ui uses a permissive canned eslint configuration.

ChRIS_ui/package.json

Line 67 in 49193ea

"lint": "eslint '*/**/*.{js,ts,tsx}' --quiet --fix",

The IDE auto-formatters of many contributors will do things like add semicolons, change quotes from single to double, ... making PR diffs hard to read and causing merge conflicts between branches.

This is a feature request for us to use a strict linter and a Github Actions workflow to check code style. Merging of PRs should only be allowed if and only if the code style check passes.

We can steal the eslint configuration from ChRIS_store_ui. See FNNDSC/ChRIS_store_ui#168 and its linked PR FNNDSC/ChRIS_store_ui#229

[Timonwa]

I will work on this.

[nanfuka]

@Timonwa can I collaborate with you on this on this issues. i have some experience there.

[Timonwa]

No problem @nanfuka. I made a comment on the pull request. Let me tag you on it.