Upgrade Stylelint 16, remove Jest & update other packages #45
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
lb-
commented
Oct 11, 2024
lb-
commented
- Remove Jest & use Node.js test runner as it fully supports ESM modules and is better suited to our testing
- Simplify Eslint config as we no longer need to have configs suitable for Jest
- Update all packages to latest possible version we can support (inc. Stylelint 16)
- Remove deprecated Stylelint SCSS rules
- Migrate code to ESM modules to suit new Stylelint 16 requirements & Node 20 without changing the output of our package to ESM at this stage
- Closes Update to support Stylelint 16 #43
|
@thibaudcolas would love to get your eyes on this please. |
- Avoid usage of Jest, no longer needed & does not handle ESM testing well - Update eslint config to avoid references to Jest and simplify JS usage
lb-
force-pushed
the
feature/43-upgrade-stylelint-16
branch
from
da8cd5d
to
3d2012a
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.