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

[FIX] Switch to maintained fork of "css" package #272

Merged
merged 2 commits into from Jan 23, 2023

Conversation

ryenus
Copy link
Contributor

@ryenus ryenus commented Jan 16, 2023

This had been first attempted in 0abe66a but later reverted by 3b6c459
due to adobe/css-tools#77, reapplying since
the issue is now fixed.


This solves the issue of having outdated and potential insecure
transitive dependencies.

There are no known behavior changes, so this is considered a
non-breaking change / fix.

See: reworkcss/css#163
See: SamVerschueren/decode-uri-component#6

This had been first attempted in 0abe66a but later reverted by 3b6c459
due to adobe/css-tools#77, reapplying since
the issue is now fixed.

---

This solves the issue of having outdated and potential insecure
transitive dependencies.

There are no known behavior changes, so this is considered a
non-breaking change / fix.

See: reworkcss/css#163
See: SamVerschueren/decode-uri-component#6
@ryenus
Copy link
Contributor Author

ryenus commented Jan 16, 2023

Ping @matz3

@RandomByte RandomByte self-requested a review January 17, 2023 09:37
@RandomByte
Copy link
Member

Thank you for your contribution. I added a commit to fix the ESLint issues and will validate the change internally. Once that is done, we'll create a new release for this 🎉

@RandomByte RandomByte merged commit 0bb592d into SAP:master Jan 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants