-
Notifications
You must be signed in to change notification settings - Fork 252
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
Update devDependencies and fix new issues #797
Conversation
We are not using modules, so we aren't in strict mode by default.
@@ -7,7 +7,7 @@ const yaml = require('js-yaml'); | |||
const SVGSpriterConfig = require('../../../lib/svg-sprite/config.js'); | |||
|
|||
jest.mock('node:fs'); | |||
jest.mock('yaml'); | |||
jest.mock('js-yaml'); |
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 don't quite get how this worked all this time 😛
@@ -324,10 +324,7 @@ module.exports = class SVGSpriterConfig { | |||
['padding'].forEach(function(property) { | |||
let spacing; | |||
|
|||
if (!Array.isArray(this.spacing[property])) { |
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.
Was this eslint's correction?
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.
Yeah, I guess the no negated condition rule I guess added some fixes
Yes. The problem with playwright-chromium still exists. I do not have much time to deal with it and I sure this is a chromium problem rather than a playwright. |
@Kreeg: I haven't updated playwright-chromium due to that old issue you were facing. I assume you are still facing it with the latest version, right?
The only other deps that are outdated is glob and pngjs. For glob there's no hurry and in fact we better keep it to v7.x for now. pngjs I didn't try to update it, so if you have time feel free to update it :)