-
-
Notifications
You must be signed in to change notification settings - Fork 598
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
ci: use Deno v2 for a test running for deno #3509
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3509 +/- ##
=======================================
Coverage 95.58% 95.58%
=======================================
Files 155 155
Lines 9357 9357
Branches 2749 2757 +8
=======================================
Hits 8944 8944
Misses 413 413 ☔ View full report in Codecov by Sentry. |
Hi @nakasyou Thank you! I've left a comment. |
I have a question. Deno 2.0 has breaking changes. Should I test both 1.x and 2.0? |
I agree. Also, Deno's LTS is shorter than Node.js, so there is no need to test multiple versions. |
Okay. I'll merge this now. Thanks! |
* chore: use Deno v2 for a test running for deno * use denoland/setup-deno@v2
Deno 2 has been released, so I changed a CI.
The author should do the following, if applicable
bun run format:fix && bun run lint:fix
to format the code