chore(deps): update dependency @vitest/eslint-plugin to v1.6.3#3645
chore(deps): update dependency @vitest/eslint-plugin to v1.6.3#3645xDivisionByZerox merged 6 commits intonextfrom
Conversation
✅ Deploy Preview for fakerjs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
9d0c15c to
5f1647c
Compare
5a59aa8 to
503aed8
Compare
1e21436 to
7e063fb
Compare
3dc09e5 to
2b906be
Compare
445dfac to
28d20f8
Compare
375fbf1 to
219b22b
Compare
1e8c006 to
64a4999
Compare
64a4999 to
c6c0172
Compare
0ece0d8 to
ce96d3c
Compare
ce96d3c to
3e4bf89
Compare
3e4bf89 to
fb271d4
Compare
This commit fixes the autofixable lint errors introduced through the vitest lint plugin update. This mostly reverts the changes introduced in #2098, where .toThrowError was replaced by .toThrow. Apprently, the plugin got it the wrong way around. For more information, look at the fixing PR in their plugin: vitest-dev/eslint-plugin-vitest#832
This change was previously not possible, since the vitest plugin was not conform with the expected eslint config typescript type. This was fixed in vitest-dev/eslint-plugin-vitest#811
As described by the comment in the config file, we require conditional logic when iterating over faker instances or doing generic test over an entire faker instance. We might want to look at vitests [test.skipIf](https://vitest.dev/api/#test-skipif) api in the future, tho.
This commit fixes warnings from the eslint rule [vitest/no-disabled-tests](https://github.com/vitest-dev/eslint-plugin-vitest/blob/main/docs/rules/no-disabled-tests.md). This is technically optional!!! I jsut wanted the lint log to look clean.
|
There where some changes required to get the vitest eslint plugin to run on the most up to date version. I've did separate commits that explain why changes where required:
Hope this helps verifying the changes. |
This commit partially reverts 03f788c because the test will not run otherwise 💀
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## next #3645 +/- ##
=======================================
Coverage 99.97% 99.97%
=======================================
Files 2995 2995
Lines 236324 236324
Branches 941 940 -1
=======================================
Hits 236267 236267
Misses 57 57 🚀 New features to boost your workflow:
|
xDivisionByZerox
left a comment
There was a problem hiding this comment.
Since I contributed the majority of changes to get the plugin to work, I'm waiting for a second approval in this dependency update PR.
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
This PR contains the following updates:
1.3.20->1.6.3Release Notes
vitest-dev/eslint-plugin-vitest (@vitest/eslint-plugin)
v1.6.3Compare Source
🚀 Features
View changes on GitHub
What's Changed
New Contributors
Full Changelog: vitest-dev/eslint-plugin-vitest@v1.6.1...v1.6.2
v1.6.1Compare Source
🐞 Bug Fixes
View changes on GitHub
What's Changed
Full Changelog: vitest-dev/eslint-plugin-vitest@v1.5.4...v1.6.0
v1.5.4Compare Source
No significant changes
View changes on GitHub
v1.5.2Compare Source
🐞 Bug Fixes
View changes on GitHub
v1.5.1Compare Source
🐞 Bug Fixes
View changes on GitHub
v1.5.0Compare Source
🚀 Features
🐞 Bug Fixes
View changes on GitHub
What's Changed
Full Changelog: vitest-dev/eslint-plugin-vitest@v1.4.4...v1.5.0
v1.4.4Compare Source
🚀 Features
require-import-vi-mockrule - by @axel7083 in #820 (abdea)🐞 Bug Fixes
test.for- by @y-hsgw in #825 (962de)View changes on GitHub
What's Changed
require-import-vi-mockrule by @axel7083 in #820test.forby @y-hsgw in #825New Contributors
Full Changelog: vitest-dev/eslint-plugin-vitest@v1.4.3...v1.4.4
v1.4.3Compare Source
🐞 Bug Fixes
View changes on GitHub
v1.4.2Compare Source
🚀 Features
require-awaited-expect-poll- by @StyleShit in #817 (ebf57)View changes on GitHub
What's Changed
require-awaited-expect-pollby @StyleShit in #817New Contributors
Full Changelog: vitest-dev/eslint-plugin-vitest@v1.4.1...v1.4.2
v1.4.1Compare Source
No significant changes
View changes on GitHub
v1.4.0Compare Source
🐞 Bug Fixes
describe.forchains - by @paescuj in #808 (97702).extendexclusive, add.scoped- by @paescuj in #809 (70708)View changes on GitHub
v1.3.26Compare Source
🐞 Bug Fixes
.for- by @paescuj in #804 (a3ab8)View changes on GitHub
v1.3.25Compare Source
🐞 Bug Fixes
View changes on GitHub
v1.3.24Compare Source
🐞 Bug Fixes
View changes on GitHub
v1.3.23Compare Source
No significant changes
View changes on GitHub
v1.3.22Compare Source
No significant changes
View changes on GitHub
Configuration
📅 Schedule: Branch creation - Between 12:00 AM and 03:59 AM, on day 1 of the month ( * 0-3 1 * * ) (UTC), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.