Skip to content

Revert the DOM query changes #42230

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

Merged
merged 3 commits into from
Jan 6, 2021
Merged

Revert the DOM query changes #42230

merged 3 commits into from
Jan 6, 2021

Conversation

orta
Copy link
Contributor

@orta orta commented Jan 6, 2021

Starts to handle the feedback in #42067

@typescript-bot typescript-bot added Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Jan 6, 2021
@orta
Copy link
Contributor Author

orta commented Jan 6, 2021

@typescript-bot run dt

@typescript-bot
Copy link
Collaborator

typescript-bot commented Jan 6, 2021

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

@orta orta added this to the TypeScript 4.2.0 milestone Jan 6, 2021
@orta
Copy link
Contributor Author

orta commented Jan 6, 2021

OK, so the biggest issue seems to be that

interface ImportMeta {
  url?: string
} 

conflicts with Nodes version which isn't nullable, I'll look into making the DOM version non-null.

@orta
Copy link
Contributor Author

orta commented Jan 6, 2021

@typescript-bot run dt

@typescript-bot
Copy link
Collaborator

typescript-bot commented Jan 6, 2021

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

@orta
Copy link
Contributor Author

orta commented Jan 6, 2021

OK, this is a big improvement - I'm going to merge this, so that it doesn't block others using the @typescript-bot run dt entirely

@typescript-bot
Copy link
Collaborator

typescript-bot commented Jan 6, 2021

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

@orta orta merged commit 35c8df0 into microsoft:master Jan 6, 2021
sandersn added a commit that referenced this pull request Jan 8, 2021
sandersn added a commit that referenced this pull request Jan 8, 2021
Zzzen pushed a commit to Zzzen/TypeScript that referenced this pull request Jan 16, 2021
* Revert the DOM query changes

* Adds baselines

* Hotfix for the removeal of the url nullabulity for testing
Zzzen pushed a commit to Zzzen/TypeScript that referenced this pull request Jan 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants