-
Notifications
You must be signed in to change notification settings - Fork 12.2k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃 User test baselines have changed for refs/heads/master #34411
馃 User test baselines have changed for refs/heads/master #34411
Conversation
fe83b15
to
1e08429
Compare
cc @sandersn |
2056ce1
to
c47dfc7
Compare
c47dfc7
to
fb539d5
Compare
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.
One regression in webpack that I think is acceptable; in JSDoc a global interface is now resolved before a local value.
lib/logging/Logger.js(117,20): error TS2345: Argument of type 'string' is not assignable to parameter of type '"error" | "time" | "group" | "debug" | "log" | "info" | "warn" | "trace" | "groupCollapsed" | "groupEnd" | "profile" | "profileEnd" | "clear" | "status"'. | ||
lib/logging/Logger.js(127,20): error TS2345: Argument of type 'string' is not assignable to parameter of type '"error" | "time" | "group" | "debug" | "log" | "info" | "warn" | "trace" | "groupCollapsed" | "groupEnd" | "profile" | "profileEnd" | "clear" | "status"'. | ||
lib/util/registerExternalSerializer.js(253,14): error TS2339: Property 'line' does not exist on type 'Position'. | ||
lib/util/registerExternalSerializer.js(254,14): error TS2339: Property 'column' does not exist on type 'Position'. |
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.
resolves to Position type, which I think is an addition to the DOM in 3.7
previously it resolved to the Position value const Position = require('acorn').Position
, which was intentionally then given the type any
because the webpack maintainers haven't had time to type acorn correctly.
lib/logging/Logger.js(127,20): error TS2345: Argument of type 'string' is not assignable to parameter of type '"error" | "time" | "group" | "debug" | "log" | "info" | "warn" | "trace" | "groupCollapsed" | "groupEnd" | "profile" | "profileEnd" | "clear" | "status"'. | ||
lib/util/registerExternalSerializer.js(253,14): error TS2339: Property 'line' does not exist on type 'Position'. | ||
lib/util/registerExternalSerializer.js(254,14): error TS2339: Property 'column' does not exist on type 'Position'. | ||
lib/validateSchema.js(15,11): error TS2339: Property 'children' does not exist on type 'ErrorObject'. |
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.
schema-utils' types appear to be wrong (or mismatching webpack's usage at least)
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.
The number of recent PRs fixing the TS types of schema-utils suggests that this is the problem.
This test run was triggerd by a request on #33716
Please review the diff and merge if no changes are unexpected.
You can view the build log here.
cc @microsoft/typescript