Skip to content

🤖 User test baselines have changed#32732

Merged
sandersn merged 1 commit into
microsoft:masterfrom
typescript-bot:user-update-Microsoft-20190702
Aug 6, 2019
Merged

🤖 User test baselines have changed#32732
sandersn merged 1 commit into
microsoft:masterfrom
typescript-bot:user-update-Microsoft-20190702

Conversation

@typescript-bot
Copy link
Copy Markdown
Collaborator

Please review the diff and merge if no changes are unexpected.
You can view the build log here.

cc @weswigham @sandersn @RyanCavanaugh

Copy link
Copy Markdown
Member

@sandersn sandersn left a comment

Choose a reason for hiding this comment

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

Changes from chrome:
Wesley's fix to nameless typedefs and enums fixes a lot of errors.

node_modules/acorn/dist/acorn.js(3035,8): error TS2339: Property 'exprAllowed' does not exist on type 'TokenType'.
node_modules/acorn/dist/acorn.js(5297,5): error TS2339: Property 'nextToken' does not exist on type 'Parser'.
node_modules/acorn/dist/acorn.js(5298,12): error TS2339: Property 'parseExpression' does not exist on type 'Parser'.
node_modules/acorn/dist/acorn_loose.es.js(1,56): error TS2440: Import declaration conflicts with local declaration of 'defaultOptions'.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

from @orta's change to allow property assignment declarations on imported things.

@@ -1,7 +0,0 @@
Exit Code: 1
Standard output:
node_modules/zone.js/dist/zone.js.d.ts(292,36): error TS2304: Cannot find name '_PatchFn'.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

seems like an update to zone.js. Its current state seems just fine.

@sandersn sandersn merged commit 024193f into microsoft:master Aug 6, 2019
@microsoft microsoft locked as resolved and limited conversation to collaborators Oct 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants