fix: eslint auto import type declaration#2297
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 #2297 +/- ##
==========================================
+ Coverage 78.43% 78.56% +0.12%
==========================================
Files 134 134
Lines 3914 3918 +4
Branches 888 888
==========================================
+ Hits 3070 3078 +8
+ Misses 753 751 -2
+ Partials 91 89 -2 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
What does this PR do? |
|
@aklinker1 fixes issue with missing type import for |
|
@aklinker1 After #2295 #2296 i'll update this branch and add testst |
|
Seems like this could be merged as-is today. |
|
Woops, didn't mean to close |
|
@aklinker1 Yeah, it can go, i can adjust other related PRs, not necessary that one. |
|
@aklinker1 I know why that's draft. I want to merge those 2 PRs and then add tests there. Let's wait till both will be merged and then i'll update that branch and add those tests. |
Overview
Fixed abandoned #1981
Blocked by #2296 #2295 (After those, i'll adjust code to version 10)
Manual Testing
Let's see if tests pass