-
Notifications
You must be signed in to change notification settings - Fork 57
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
Issue with winston dependency breaking builds relying on cloud-sdk/util #206
Comments
Hey @edonadei, I locked the version to 3.3.0. We will probably release a new version tomorrow or Wednesday. You will have to upgrade to the latest version of the SDK though, which will then be 1.22.0. As you are using a very old version, consider the migration guide. |
There is a fixed version available now: winstonjs/winston#1814 (comment) |
I checked the code, the reference to git was introduced in v3.3.1 and fixed in v3.3.2, so theoretically your builds should work without any action now. If there is still something broken, maybe it is related to something else? |
Yeah sorry that was a misfire on my part. It worked with npm install + resolution, but our preinstall resolution script was ignored during an explicit npm install of the module inside of our pipeline. So yes, only 3.3.1 was affected and our builds are running fine after the release of 3.3.2 without any action on our side (since the sdk defined "winston@^3.2.1"). |
Describe the bug
Winston dependencies break multiples jenkins builds since the update 3.3.0/3.3.1 to winstonjs/winston#1814
Expected behavior
We should use the cloud-sdk dependency without errors from this dependency
It is actually
"winston": "^3.2.1"
but it should be"winston": "3.2.1"
or"winston": "~3.2.1"
Used Versions:
The text was updated successfully, but these errors were encountered: