Skip to content

Conversation

@m-brophy
Copy link
Contributor

@m-brophy m-brophy commented May 10, 2024

@m-brophy m-brophy added Ready to Review auto-backport Set the PR for being backported auto-backport-to-releases/6.3.z Backport this issue to releases/6.3.z branch labels May 10, 2024
@m-brophy m-brophy requested a review from carlosthe19916 May 10, 2024 09:59
"proxy": "http://localhost:8080",
"homepage": "./",
"dependencies": {
"@adobe/css-tools": "^4.3.3",
Copy link
Member

Choose a reason for hiding this comment

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

As this is not a direct dependency for the UI. It is better to set this line under the resolutions section of the package.json

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this line has been moved as suggested

Copy link
Member

@carlosthe19916 carlosthe19916 left a comment

Choose a reason for hiding this comment

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

Same comment as windup/windup#1739

Copy link
Member

@carlosthe19916 carlosthe19916 left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @m-brophy

@m-brophy m-brophy merged commit b136f5a into windup:master May 10, 2024
github-actions bot pushed a commit that referenced this pull request May 10, 2024
* WINDUP-4115 CVE upgrade css-tools

* WINDUP-4115 place css-tools in resolutions section

(cherry picked from commit b136f5a)
@github-actions
Copy link

💚 All backports created successfully

Status Branch Result
releases/6.3.z

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

m-brophy added a commit that referenced this pull request May 10, 2024
* WINDUP-4115 CVE upgrade css-tools

* WINDUP-4115 place css-tools in resolutions section

(cherry picked from commit b136f5a)

Co-authored-by: Mark Brophy <[email protected]>
@m-brophy m-brophy deleted the WINDUP-4155 branch May 10, 2024 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Set the PR for being backported auto-backport-to-releases/6.3.z Backport this issue to releases/6.3.z branch Ready to Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants