Skip to content
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 warnings #29

Merged
merged 4 commits into from
Mar 8, 2024
Merged

Conversation

gentlementlegen
Copy link
Member

Resolves #28

@gentlementlegen gentlementlegen requested a review from 0x4007 March 8, 2024 11:08
Copy link
Contributor

github-actions bot commented Mar 8, 2024

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

Copy link
Contributor

github-actions bot commented Mar 8, 2024

Lines Statements Branches Functions
Coverage: 80%
80% (4/5) 100% (0/0) 66.66% (2/3)

JUnit

Tests Skipped Failures Errors Time
1 0 💤 0 ❌ 0 🔥 2.077s ⏱️
Coverage Report (80%)
File% Stmts% Branch% Funcs% LinesUncovered Line #s
All files8010066.6680 
   main.ts8010066.66809

@ubiquity-os-deployer
Copy link

@gentlementlegen
Copy link
Member Author

gentlementlegen commented Mar 8, 2024

@pavlovcik The results from Knip seem to be empty and clean yet it is displayed as if there were errors in the CI. I will merge and see if it is just an error because the previous tests failed, otherwise will fix it in a subsequent PR.

https://github.com/ubiquity/ts-template/pull/29/checks?check_run_id=22433462903

@gentlementlegen gentlementlegen merged commit 7695c37 into ubiquity:development Mar 8, 2024
3 of 5 checks passed
@gentlementlegen gentlementlegen deleted the fix/knip branch March 8, 2024 11:24
@ubiquibot ubiquibot bot mentioned this pull request Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Knip Report
2 participants