Skip to content
This repository has been archived by the owner on Sep 19, 2022. It is now read-only.

Höhere Version von hoek wegen Securitywarnung. #37

Merged
merged 2 commits into from
Feb 11, 2022

Conversation

Eymux
Copy link
Contributor

@Eymux Eymux commented May 8, 2018

No description provided.

@Eymux Eymux requested a review from sikelerd May 22, 2018 13:55
Copy link
Contributor

@sikelerd sikelerd left a comment

Choose a reason for hiding this comment

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

Ich finde es unschön wenn wir deshalb eine neue explizite Dependency deklarieren. Potentiell müssen wir das später für immer mehr Dependencies machen. Ich würde lieber das eigentliche Paket aktualisieren sobald dieses aktualisiert wurde.

@Eymux
Copy link
Contributor Author

Eymux commented Jul 2, 2018

hoek wird von @angular/compiler-cli und node-sass verwendet. So weit ich das feststellen kann haben beide Projekte das Problem nicht gefixt, so dass ein Upgrade nichts bringt.
s.a. angular/angular-cli#10827 und sass/node-sass#2355

@sikelerd
Copy link
Contributor

sikelerd commented Jul 9, 2018

Für node-sass gibt es jetzt einen Fix in der Version 4.9.1 (siehe sass/node-sass#2355)

@sikelerd
Copy link
Contributor

sikelerd commented Jul 9, 2018

Braucht man node-sass überhaupt? Kann die dependency komplett entfernt werden?

@bjoernranft bjoernranft merged commit 874647c into WollMux:master Feb 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants