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
Yarn upgrade archiver and logging #1379
Conversation
@@ -13,7 +13,7 @@ export const frontController = (req: Request, res: Response) => { | |||
req.params.version, | |||
isPreview, | |||
) | |||
const edition = req.params.edition | |||
const edition = getEditionOrFallback(req.params.edition) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this is just a yarn upgrade
then why do we have changes to logic?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was wondering that! Basically after I ran yarn upgrade the validate
step started failing - perhaps because of babel changes? Doesn't really make sense though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can see the failure stuff here - basically it was angry about treating strings as typed values
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very strange, as long as you have tested it, then im happy for it to go in.
3310615
to
1aa14d5
Compare
1aa14d5
to
bf5ed19
Compare
This is another effort to resolve the lodash vulnerability addressed in #1376 - I've run
yarn upgrade
on archiver and logging