feat!: Add eslint 10 support for unimport of wxt#2295
Conversation
✅ Deploy Preview for creative-fairy-df92c4 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
@wxt-dev/analytics
@wxt-dev/auto-icons
@wxt-dev/browser
@wxt-dev/i18n
@wxt-dev/is-background
@wxt-dev/module-react
@wxt-dev/module-solid
@wxt-dev/module-svelte
@wxt-dev/module-vue
@wxt-dev/runner
@wxt-dev/storage
@wxt-dev/unocss
@wxt-dev/webextension-polyfill
wxt
commit: |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2295 +/- ##
==========================================
- Coverage 79.73% 79.71% -0.03%
==========================================
Files 131 131
Lines 3804 3805 +1
Branches 861 862 +1
==========================================
Hits 3033 3033
- Misses 686 687 +1
Partials 85 85 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@aklinker1 Should be good right now :) |
|
This PR is a breaking change as it stands now. I'm not going to merge it, we're making way to many breaking changes. Make it work for people who still use |
|
@aklinker1 Ok, i'll do a fallback and use |
unimport of wxtunimport of wxt
|
@aklinker1 I've rethough and rereview it, and i won't adjust it, because it can go to Let's wait with those(3) PRs on that topic, till you'll be releasing next It isn't that big breaking changes, ok yeah, user need to change params, but that's it. I want to suggest to do it in that way. |
|
No, this does not require a breaking change. We cannot have a release with as many breaking changes as this upcoming one ever again. I was too lax about this, and I don't think it will go over well with developers. Deprecate the options and continue to support them. We can remove them in the major version after this one. Removing deprecated APIs has a much smaller overhead for devs compared to hard cutting to a new behavior - people get warnings and can see the migration steps and take them ahead of time. |
Overview
Added version 10 of
eslinttoEslintrcandResolvedEslintrctypesRelated with #1981