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

[Theia] Upgrade version of less to 3.0.3 for fixing the vulnerability in cryptiles package #4441

Merged
merged 1 commit into from
Feb 28, 2019
Merged

[Theia] Upgrade version of less to 3.0.3 for fixing the vulnerability in cryptiles package #4441

merged 1 commit into from
Feb 28, 2019

Conversation

vinayb21
Copy link
Contributor

@vinayb21 vinayb21 commented Feb 28, 2019

Fixes: #4440

@@ -41,7 +41,7 @@
"font-awesome-webpack": "0.0.5-beta.2",
"fs-extra": "^4.0.2",
"ignore-loader": "^0.1.2",
"less": "^2.7.2",
"less": "^3.0.3",
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Not now. We decided to not hold back such changes until the foundation has decided how to go forward.

@akosyakov
Copy link
Member

@vinayb21 please commit changes to yarn.lock file

@kittaakos
Copy link
Contributor

kittaakos commented Feb 28, 2019

@vinayb21, FYI the build has failed because of this #3787.

Update: I have restarted the Linux build.

@vinayb21
Copy link
Contributor Author

@vinayb21, FYI the build has failed because of this #3787.

Update: I have restarted the Linux build.

@kittaakos The build failed.

@kittaakos
Copy link
Contributor

Having the macOS and the Windows green builds, I think it is safe to merge this PR. But I leave this decision for the reviewer :)

@@ -41,7 +41,7 @@
"font-awesome-webpack": "0.0.5-beta.2",
"fs-extra": "^4.0.2",
"ignore-loader": "^0.1.2",
"less": "^2.7.2",
"less": "^3.0.3",
Copy link
Contributor

Choose a reason for hiding this comment

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

Not now. We decided to not hold back such changes until the foundation has decided how to go forward.

@svenefftinge svenefftinge merged commit b1ef055 into eclipse-theia:master Feb 28, 2019
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.

4 participants