Skip to content

Remove old test262 and dt runner infra #53125

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 6 commits into from
Mar 7, 2023

Conversation

jakebailey
Copy link
Member

@jakebailey jakebailey commented Mar 7, 2023

Follow-up to #53118.

I'll try and get test262 going in the error deltas repo, at the very least to just check parsing. Done.

@typescript-bot typescript-bot added Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Mar 7, 2023
@jakebailey jakebailey changed the title Remove docker and user tests, infra Remove old test262 and dt runner infra Mar 7, 2023
@jakebailey
Copy link
Member Author

I thought I could delete the scripts, but they actually are for importing DT into RWC, so I'll delete that another day.

@jakebailey
Copy link
Member Author

I have absolute no clue why the linter is complaining...

@jakebailey
Copy link
Member Author

jakebailey commented Mar 7, 2023

If anyone watching this is a genius, see if you can figure out why that previous commit makes ts-eslint work again... I don't get it.

@jakebailey
Copy link
Member Author

Well, I "figured" it out, in that I assumed that it had to do with ts-eslint loading all code into one Program and there being oddities with type references, so I just made a new eslint-only config which adds them unconditionally.

@jakebailey
Copy link
Member Author

microsoft/typescript-error-deltas#102 for the error deltas change. Surprise, we debug assert, so that's something to look into.

@jakebailey jakebailey merged commit a6be79d into microsoft:main Mar 7, 2023
@jakebailey jakebailey deleted the remove-test262-dt branch March 7, 2023 19:59
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.

3 participants