Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update dev dependencies stylelint SASS #4173

Merged
merged 5 commits into from Feb 4, 2022

Conversation

Alkarex
Copy link
Member

@Alkarex Alkarex commented Jan 24, 2022

Major update for stylelint with breaking changes
https://stylelint.io/migration-guide/to-14/

Applied automatic SASS migration to current syntax
https://sass-lang.com/documentation/cli/migrator#migrations

And a few manual fixes such as:
https://github.com/stylelint-scss/stylelint-scss/blob/master/src/rules/dollar-variable-no-missing-interpolation/README.md

Other dev dependencies are minor updates, including fixing a security vulnerability #4168

Major update for stylelint with breaking changes
https://stylelint.io/migration-guide/to-14/

Applied automatic SASS migration to current syntax
https://sass-lang.com/documentation/cli/migrator#migrations

And a few manual fixes such as:
https://github.com/stylelint-scss/stylelint-scss/blob/master/src/rules/dollar-variable-no-missing-interpolation/README.md

Other dev dependencies are minor updates.
@Alkarex Alkarex added Theme System care Everything related to the system care labels Jan 24, 2022
@Alkarex Alkarex added this to the 1.20.0 milestone Jan 24, 2022
@Alkarex Alkarex requested a review from math-GH January 24, 2022 21:05
Makefile Outdated Show resolved Hide resolved
Copy link
Member

@Frenzie Frenzie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine, didn't check the SCSS stuff in detail but the fact that the outcome is identical probably speaks for itself.

p/themes/Mapco/mapco.css Show resolved Hide resolved
@Alkarex Alkarex merged commit 5a891dc into FreshRSS:edge Feb 4, 2022
@Alkarex Alkarex deleted the update-stylelint branch February 4, 2022 14:41
@Alkarex Alkarex mentioned this pull request Feb 4, 2022
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
System care Everything related to the system care Theme
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants