-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix knip issues #9
Fix knip issues #9
Conversation
…iguration chore: add correct Knip workflow configuration
Remove unused dependencies, set public tag for public exports and add unused files to ignore list.
JUnit
Coverage Report (81%)
|
Knip green, QA also done in my forked repository: gitcoindev#2 I noticed some spelling errors thrown by cspell, give me a minute and I will fix them in one go. |
Knip green again, spelling issues fixed, pr ready for the review. |
Interestingly Knip detects |
thanks for the PR, looks great |
let me review accordingly next time lol @gitcoindev |
Sure, I will! |
fix: updated Jest test comment
Knip QA