Skip to content

Conversation

serhalp
Copy link
Member

@serhalp serhalp commented Feb 20, 2025

Summary

Initial list found via https://knip.dev, then manually verified each and pruned false positives

@serhalp serhalp force-pushed the fix/remove-unused-deps branch from 86fead8 to 1d9d59c Compare February 25, 2025 15:16
Copy link
Contributor

This pull request adds or modifies JavaScript (.js, .cjs, .mjs) files.
Consider converting them to TypeScript.

Copy link
Contributor

This pull request adds or modifies JavaScript (.js, .cjs, .mjs) files.
Consider converting them to TypeScript.

@serhalp serhalp force-pushed the fix/remove-unused-deps branch from ed7ab16 to 865dfe1 Compare March 12, 2025 21:03
Copy link
Contributor

This pull request adds or modifies JavaScript (.js, .cjs, .mjs) files.
Consider converting them to TypeScript.

@serhalp serhalp force-pushed the fix/remove-unused-deps branch from 865dfe1 to c207d57 Compare March 12, 2025 21:05
Copy link
Contributor

This pull request adds or modifies JavaScript (.js, .cjs, .mjs) files.
Consider converting them to TypeScript.

@serhalp serhalp force-pushed the fix/remove-unused-deps branch 2 times, most recently from d8688dc to 18ca7a2 Compare March 18, 2025 19:31
@serhalp serhalp force-pushed the fix/remove-unused-deps branch from 7b36b7c to 0847768 Compare March 19, 2025 16:24
@serhalp serhalp marked this pull request as ready for review March 19, 2025 16:56
@serhalp serhalp requested a review from a team as a code owner March 19, 2025 16:56
"tsd": "^0.31.0",
"vitest": "^0.34.0",
"yarn": "^1.22.4"
"yarn": "^1.22.22"
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😓 This is used in a fixture used by a couple tests. I didn't see an easy way to improve it so I kept it.

@serhalp serhalp merged commit 99ec56a into main Mar 19, 2025
33 checks passed
@serhalp serhalp deleted the fix/remove-unused-deps branch March 19, 2025 17:14
This was referenced Sep 2, 2025
This was referenced Sep 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants