Skip to content

🤖 User test baselines have changed for refs/heads/master #36382

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

Conversation

typescript-bot
Copy link
Collaborator

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

@typescript-bot typescript-bot force-pushed the user-update-Microsoft-20200004-refs/heads/master branch from 8332bae to d0ac227 Compare January 23, 2020 21:45
Copy link
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.

Not sure what is going on with sift, but everything else looks fine.

@@ -0,0 +1,7 @@
Exit Code: 1
Standard output:
node_modules/sift/index.d.ts(80,15): error TS2307: Cannot find module './core'.
Copy link
Member

Choose a reason for hiding this comment

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

oh, they fixed a typo and commented out the entire index.d.ts. I'll see if I can get a PR together to fix it.

Copy link
Member

Choose a reason for hiding this comment

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

🤦‍♂ "fix typo" secretly deletes feature

Copy link
Member

Choose a reason for hiding this comment

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

"fix typo" is my new catch-all commit message.

@sandersn
Copy link
Member

I'm going to merge this while sift sorts out its packaging.

@sandersn sandersn merged commit 3b919e2 into microsoft:master Jan 23, 2020
@sandersn
Copy link
Member

crcn/sift.js#174 should fix sift when it ships

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants