Replace unimported with knip

[dmijatovic]

Replace unimported with knip

Closes #1273

Changes proposed in this pull request:

• Replace "retired" unimported with knip.

How to test:

• This is specific frontend "feature" to check for unimported files, unused types, unused dependencies and some other "quality" checks. It is not integrated in the CI/CD, rather it is executed manually from time to time. To test you will need to have Node and NPM installed on your machine.
• Navigate to frontend folder and run npm install to install all dependencies. You might also remove old node_modules folder using rm -rf node_modules before running npm install
• npm run knip:files to check for unimported/unused files
• npm run knip:types to check for unused exported type definitions
• Both checks should produce 0 in this PR because I removed unused files and types.
• npm test to run unit tests locally. All test should pass.
• npm run build to build frontend. The build should be successful.
• make start to build app and create test data. The app should work without problems.

PR Checklist:

• [ ] Increase version numbers in docker-compose.yml
• [ ] Link to a GitHub issue
• [ ] Update documentation
• [ ] Tests

[dmijatovic]

@ewan-escience Thanks. I'll fix typescript version, it seem to be changed during install of knip. Concerning the exports "feature", I tried it but do not see added value at the moment. The exported functions are used in the file itself by some other functions. The reason that these functions are exported is to indicate that these can be used from other location, so the functions are not necessary the private functions.

[sonarcloud[bot]]

Quality Gate passed for 'rsd-frontend'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud