Skip to content
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

Add console types #37048

Closed
wants to merge 2 commits into from
Closed

Add console types #37048

wants to merge 2 commits into from

Conversation

sandersn
Copy link
Member

From microsoft/TypeScript-DOM-lib-generator#452

Test diffs look OK, so I'm going to run user/docker/RWC/DT tests

Test diffs look OK, so I'm going to run user/docker/RWC/DT tests
@sandersn
Copy link
Member Author

@typescript-bot user test this
@typescript-bot run dt
@typescript-but test this

@typescript-bot
Copy link
Collaborator

typescript-bot commented Feb 26, 2020

Heya @sandersn, I've started to run the parallelized community code test suite on this PR at a44abbc. You can monitor the build here.

@typescript-bot
Copy link
Collaborator

typescript-bot commented Feb 26, 2020

Heya @sandersn, I've started to run the parallelized Definitely Typed test suite on this PR at a44abbc. You can monitor the build here.

@sandersn
Copy link
Member Author

@typescript-bot test this

@typescript-bot
Copy link
Collaborator

typescript-bot commented Feb 26, 2020

Heya @sandersn, I've started to run the extended test suite on this PR at a44abbc. You can monitor the build here.

@sandersn
Copy link
Member Author

@typescript-bot user test this
@typescript-bot run dt
@typescript-bot test this

@typescript-bot
Copy link
Collaborator

typescript-bot commented Feb 26, 2020

Heya @sandersn, I've started to run the extended test suite on this PR at bb4ca96. You can monitor the build here.

@typescript-bot
Copy link
Collaborator

typescript-bot commented Feb 26, 2020

Heya @sandersn, I've started to run the parallelized community code test suite on this PR at bb4ca96. You can monitor the build here.

@typescript-bot
Copy link
Collaborator

typescript-bot commented Feb 26, 2020

Heya @sandersn, I've started to run the parallelized Definitely Typed test suite on this PR at bb4ca96. You can monitor the build here.

@sandersn
Copy link
Member Author

@weswigham looks like both user tests and RWC tests fail. user tests run and then fail early on in the upload with

fatal: couldn't find remote ref refs/heads/37048/head
error: pathspec 'FETCH_HEAD' did not match any file(s) known to git

RWC tests fail when trying to open the PR with:

Branch 'rwc-pr-37048' set up to track remote branch 'rwc-pr-37048' from 'origin'.
Error: TF401179: An active pull request for the source and target branch already exists.
    at RestClient.<anonymous> (/home/vsts/work/1/s/internal/scripts/tsscripts/node_modules/typed-rest-client/RestClient.js:173:28)
    at Generator.next (<anonymous>)
    at fulfilled (/home/vsts/work/1/s/internal/scripts/tsscripts/node_modules/typed-rest-client/RestClient.js:6:58)
    at process._tickCallback (internal/process/next_tick.js:68:7)

So I guess this already opened a PR, but failed to switch over to pushing commits.

@weswigham
Copy link
Member

Ah, I typo'd my change to the user test script in devops to retarget the PR to the branch head (it should be refs/pulls, not refs/heads).

The RWC thing is normal so long as an RWC diff PR already exists for the PR; that RWC diff PR still gets a forced update (and since the build itself already failed, I don't really consider the nonzero exit of the pr script to be an issue)

@weswigham
Copy link
Member

The user run should now be using the correct ref name for the PR phase~

@sandersn
Copy link
Member Author

Using the RWC run from 10:30, I see that:

  1. Angular 2 used profile and profileEnd, and an MS internal project does too.
  2. Assumes that Window has a property console, except that they use a custom subclass of Window so they don't get the window: Window & typeof globalThis.
  3. Another one, but just has a type reference to Window.
  4. Uses Console as a type. (these last two are in introspective code that is probably uncommon)
  5. core-js tries to extend the Console type.

The main surprise for me was (1). (2) - (5) are definitely breaking changes, even though they are fairly rare.

@sandersn sandersn closed this Mar 25, 2020
@sandersn sandersn deleted the add-console-types branch March 25, 2020 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants