sveltejs / svelte-eslint-parser

Svelte parser for ESLint
https://sveltejs.github.io/svelte-eslint-parser/
MIT License
90 stars 19 forks source link

chore(deps): update dependency chai to v5 #458

Open renovate[bot] opened 9 months ago

renovate[bot] commented 9 months ago

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
chai (source) ^4.5.0 -> ^5.0.0 age adoption passing confidence
@types/chai (source) ^4.3.19 -> ^5.0.0 age adoption passing confidence

Release Notes

chaijs/chai (chai) ### [`v5.1.1`](https://redirect.github.com/chaijs/chai/releases/tag/v5.1.1) [Compare Source](https://redirect.github.com/chaijs/chai/compare/v5.1.0...v5.1.1) #### What's Changed - Set up ESLint for JSDoc comments by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1605](https://redirect.github.com/chaijs/chai/pull/1605) - build(deps-dev): bump ip from 1.1.8 to 1.1.9 by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/chaijs/chai/pull/1608](https://redirect.github.com/chaijs/chai/pull/1608) - Correct Mocha import instructions by [@​MattiSG](https://redirect.github.com/MattiSG) in [https://github.com/chaijs/chai/pull/1611](https://redirect.github.com/chaijs/chai/pull/1611) - fix: support some virtual contexts in `toThrow` by [@​43081j](https://redirect.github.com/43081j) in [https://github.com/chaijs/chai/pull/1609](https://redirect.github.com/chaijs/chai/pull/1609) #### New Contributors - [@​MattiSG](https://redirect.github.com/MattiSG) made their first contribution in [https://github.com/chaijs/chai/pull/1611](https://redirect.github.com/chaijs/chai/pull/1611) **Full Changelog**: https://github.com/chaijs/chai/compare/v5.1.0...v5.1.1 ### [`v5.1.0`](https://redirect.github.com/chaijs/chai/releases/tag/v5.1.0) [Compare Source](https://redirect.github.com/chaijs/chai/compare/v5.0.3...v5.1.0) #### What's Changed - Remove useless guards and add parentheses to constuctors by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1593](https://redirect.github.com/chaijs/chai/pull/1593) - Cleanup jsdoc comments by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1596](https://redirect.github.com/chaijs/chai/pull/1596) - Convert comments in "legal comments" format to jsdoc or normal comments by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1598](https://redirect.github.com/chaijs/chai/pull/1598) - Implement `iterable` assertion by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1592](https://redirect.github.com/chaijs/chai/pull/1592) - Assert interface fix by [@​developer-bandi](https://redirect.github.com/developer-bandi) in [https://github.com/chaijs/chai/pull/1601](https://redirect.github.com/chaijs/chai/pull/1601) - Set support in same members by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1583](https://redirect.github.com/chaijs/chai/pull/1583) - Fix publish script by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1602](https://redirect.github.com/chaijs/chai/pull/1602) #### New Contributors - [@​developer-bandi](https://redirect.github.com/developer-bandi) made their first contribution in [https://github.com/chaijs/chai/pull/1601](https://redirect.github.com/chaijs/chai/pull/1601) **Full Changelog**: https://github.com/chaijs/chai/compare/v5.0.3...v5.1.0 ### [`v5.0.3`](https://redirect.github.com/chaijs/chai/releases/tag/v5.0.3) [Compare Source](https://redirect.github.com/chaijs/chai/compare/v5.0.2...v5.0.3) Fix bad v5.0.2 publish. **Full Changelog**: https://github.com/chaijs/chai/compare/v5.0.2...v5.0.3 ### [`v5.0.2`](https://redirect.github.com/chaijs/chai/releases/tag/v5.0.2) [Compare Source](https://redirect.github.com/chaijs/chai/compare/v5.0.0...v5.0.2) #### What's Changed - build(deps): bump nanoid and mocha by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/chaijs/chai/pull/1558](https://redirect.github.com/chaijs/chai/pull/1558) - remove `bump-cli` by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1559](https://redirect.github.com/chaijs/chai/pull/1559) - Update developer dependencies by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1560](https://redirect.github.com/chaijs/chai/pull/1560) - fix: removes `??` for node compat (5.x) by [@​43081j](https://redirect.github.com/43081j) in [https://github.com/chaijs/chai/pull/1576](https://redirect.github.com/chaijs/chai/pull/1576) - Update `loupe` to latest version by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1579](https://redirect.github.com/chaijs/chai/pull/1579) - Re-enable some webkit tests by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1580](https://redirect.github.com/chaijs/chai/pull/1580) - Remove a bunch of if statements in test/should.js by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1581](https://redirect.github.com/chaijs/chai/pull/1581) - Remove a bunch of unused files by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1582](https://redirect.github.com/chaijs/chai/pull/1582) - Fix 1564 by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1566](https://redirect.github.com/chaijs/chai/pull/1566) **Full Changelog**: https://github.com/chaijs/chai/compare/v5.0.1...v5.0.2 ### [`v5.0.0`](https://redirect.github.com/chaijs/chai/releases/tag/v5.0.0) [Compare Source](https://redirect.github.com/chaijs/chai/compare/v4.5.0...v5.0.0) #### BREAKING CHANGES - Chai now only supports EcmaScript Modules (ESM). This means your tests will need to either have `import {...} from 'chai'` or `import('chai')`. `require('chai')` will cause failures in nodejs. If you're using ESM and seeing failures, it may be due to a bundler or transpiler which is incorrectly converting import statements into require calls. - Dropped support for Internet Explorer. - Dropped support for NodeJS < 18. - Minimum supported browsers are now Firefox 100, Safari 14.1, Chrome 100, Edge 100. Support for browsers prior to these versions is "best effort" (bug reports on older browsers will be assessed individually and may be marked as wontfix). #### What's Changed - feat: use chaijs/loupe for inspection by [@​pcorpet](https://redirect.github.com/pcorpet) in [https://github.com/chaijs/chai/pull/1401](https://redirect.github.com/chaijs/chai/pull/1401) - docs: fix URL in README by [@​Izzur](https://redirect.github.com/Izzur) in [https://github.com/chaijs/chai/pull/1413](https://redirect.github.com/chaijs/chai/pull/1413) - Remove `get-func-name` dependency by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1416](https://redirect.github.com/chaijs/chai/pull/1416) - Convert Makefile script to npm scripts by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1424](https://redirect.github.com/chaijs/chai/pull/1424) - Clean up README badges by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1422](https://redirect.github.com/chaijs/chai/pull/1422) - fix: package.json - deprecation warning on exports field by [@​stevenjoezhang](https://redirect.github.com/stevenjoezhang) in [https://github.com/chaijs/chai/pull/1400](https://redirect.github.com/chaijs/chai/pull/1400) - fix: deep-eql bump package to support symbols by [@​snewcomer](https://redirect.github.com/snewcomer) in [https://github.com/chaijs/chai/pull/1458](https://redirect.github.com/chaijs/chai/pull/1458) - ES module conversion PoC by [@​43081j](https://redirect.github.com/43081j) in [https://github.com/chaijs/chai/pull/1498](https://redirect.github.com/chaijs/chai/pull/1498) - chore: drop commonjs support by [@​43081j](https://redirect.github.com/43081j) in [https://github.com/chaijs/chai/pull/1503](https://redirect.github.com/chaijs/chai/pull/1503) - Update pathval by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1527](https://redirect.github.com/chaijs/chai/pull/1527) - Update check-error by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1528](https://redirect.github.com/chaijs/chai/pull/1528) - update `deep-eql` to latest version by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1542](https://redirect.github.com/chaijs/chai/pull/1542) - Inline `type-detect` as a simple function by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1544](https://redirect.github.com/chaijs/chai/pull/1544) - Update loupe by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1545](https://redirect.github.com/chaijs/chai/pull/1545) - Typo 'Test an object' not 'Test and object' by [@​mavaddat](https://redirect.github.com/mavaddat) in [https://github.com/chaijs/chai/pull/1460](https://redirect.github.com/chaijs/chai/pull/1460) - Update `assertion-error` to it's latest major version! by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1543](https://redirect.github.com/chaijs/chai/pull/1543) - Replacing Karma with Web Test Runner by [@​koddsson](https://redirect.github.com/koddsson) in [https://github.com/chaijs/chai/pull/1546](https://redirect.github.com/chaijs/chai/pull/1546) #### New Contributors - [@​Izzur](https://redirect.github.com/Izzur) made their first contribution in [https://github.com/chaijs/chai/pull/1413](https://redirect.github.com/chaijs/chai/pull/1413) - [@​stevenjoezhang](https://redirect.github.com/stevenjoezhang) made their first contribution in [https://github.com/chaijs/chai/pull/1400](https://redirect.github.com/chaijs/chai/pull/1400) - [@​43081j](https://redirect.github.com/43081j) made their first contribution in [https://github.com/chaijs/chai/pull/1498](https://redirect.github.com/chaijs/chai/pull/1498) **Full Changelog**: https://github.com/chaijs/chai/compare/4.3.1...v5.0.0

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Enabled.

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 these updates again.



This PR was generated by Mend Renovate. View the repository job log.

changeset-bot[bot] commented 9 months ago

⚠️ No Changeset found

Latest commit: 313a1897a195b6271e8963b60b6779e2077829fe

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

baseballyama commented 8 months ago

😇

Chai now only supports EcmaScript Modules (ESM).

https://github.com/chaijs/chai/releases/tag/v5.0.0

What we will do @ota-meshi ? Maybe we can use mocha as same as eslint-plugin-vue?

ESLint itself still seems to be using v4. https://github.com/eslint/eslint/blob/60b966b6861da11617ddc15487bd7a51c584c596/package.json#L112

ota-meshi commented 8 months ago

I don't think we can use ESM-only packages now. If we do make this a dual package with ESM in the future, I don't think we can drop ESM until ESLint removes support for legacy configs (legacy configs will be deprecated in ESLint v9) . However, I think there is a possibility that the test could be devised to test only the ESM.

baseballyama commented 8 months ago

I may have understood it wrong to begin with, but I understood that ESLint doesn’t support ESM and must be built with commonjs. Therefore, I understood that the test runner must also support commonjs. Since mocha seemed to support commonjs, I thought this might be one of the solutions. ESLint v9 still use commonjs, so we can't use ESM, can we?