Knip Report

[0x4007]

Handle all of these either by removing them (ideal) or by ignoring them etc if they must be. For example, the eslint stuff probably needs to be ignored since its not used at runtime, but instead, during commits in development.

---

Unused devDependencies (5)

Filename	devDependencies
package.json	@mswjs/data @types/jest eslint-config-prettier eslint-plugin-prettier ts-node

Unlisted dependencies (1)

Filename	unlisted
tests/main.test.ts	@jest/globals

Originally posted by @github-actions[bot] in #27 (comment)

[gentlementlegen]

/start

[ubiquibot]

Deadline	Fri, Mar 8, 10:21 AM UTC
Registered Wallet	0x0fC1b909ba9265A846b82CF4CE352fc3e7EeB2ED

Tips:

• Use /wallet 0x0000...0000 if you want to update your registered payment wallet address.
• Be sure to open a draft pull request as soon as possible to communicate updates on your progress.
• Be sure to provide timely updates to us when requested, or you will be automatically unassigned from the task.

[gentlementlegen]

@pavlovcik So to address all of the issues:

• Jest and Eslint related messages can be safely ignored according to the docs https://knip.dev/guides/handling-issues#eslint--jest
• ts-node is still needed by jest to understand the .ts config, and there seem to be no workaround for this except having a js file
• @mswjs/data is used in the code but ignored by knip because it is located in the mocks folder

What I did in the end is adding all of them to the ignored dependencies, I don't see a better solution at least for now.

Another option would be to use the new eslint flat configuration. But this requires to have a js file with the config which I know you don't like: https://github.com/prettier/eslint-plugin-prettier?tab=readme-ov-file#configuration-new-eslintconfigjs

[0x4007]

Okay so regarding ts-node can we do a json config?

[gentlementlegen]

@pavlovcik Just changed the configuration of Jest to be done through a JSON file.

[ubiquibot]

+ Evaluating results. Please wait...

[ubiquibot]

[ 30 WXDAI ]

@pavlovcik

Contributions Overview

View	Contribution	Count	Reward
Issue	Specification	1	27.8
Issue	Comment	1	2.2

Conversation Incentives

Comment	Formatting	Relevance	Reward
Handle all of these either by removing them (ideal) or by ignori...	27.8 h3: count: 2 score: "2" words: 6 code: count: 6 score: "6" words: 14 td: count: 4 score: "4" words: 44 hr: count: 1 score: "1" words: 0	1	27.8
Okay so regarding ts-node can we do a json config?...	2.2	0.8	2.2

[ 87.3 WXDAI ]

@FernandVEYRIER

Contributions Overview

View	Contribution	Count	Reward
Issue	Task	1	50
Issue	Comment	2	0
Issue	Comment	2	25.7
Review	Comment	1	5.8
Review	Comment	1	5.8

Conversation Incentives

Comment	Formatting	Relevance	Reward
@pavlovcik So to address all of the issues: - Jest and Eslint r...	- li: count: 3 score: "0" words: 76 code: count: 7 score: "0" words: 9	0.87	-
@pavlovcik Just changed the configuration of Jest to be done thr...	-	0.63	-
@pavlovcik So to address all of the issues: - Jest and Eslint r...	24.3 li: count: 3 score: "3" words: 76 code: count: 7 score: "7" words: 9	0.87	24.3
@pavlovcik Just changed the configuration of Jest to be done thr...	1.4	0.63	1.4
@pavlovcik The results from Knip seem to be empty and clean yet ...	5.8	0.35	5.8
@pavlovcik The results from Knip seem to be empty and clean yet ...	5.8	0.35	5.8