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

Migrate to eslint-9 flat config, typescript 5, typescript-eslint strictTypeChecked #476

Merged
merged 22 commits into from
Jul 22, 2024

Conversation

Gnuxie
Copy link
Member

@Gnuxie Gnuxie commented Jul 9, 2024

No description provided.

@Gnuxie Gnuxie changed the title Migrate to eslint-9, typescript 5, typescript-eslint strictTypeChecked Migrate to eslint-9 flat config, typescript 5, typescript-eslint strictTypeChecked Jul 9, 2024
@Gnuxie Gnuxie force-pushed the gnuxie/eslint-9-ts-5 branch from d16cd64 to 243fddf Compare July 9, 2024 17:04
Gnuxie added 8 commits July 10, 2024 15:17
The command handler used to always only return the command
without the prefix due to an operator precedence bug.
This meant that when we made the order of operations explicit,
we were now including the prefix of the command in the copy.
So when we parsed arguments the code wasn't expecting the prefix
to be there.
MPS 0.23.0 was bugged because we didn't enable
`noUncheckedIndexedAccess` while upgrading to typescript 5.
@Gnuxie Gnuxie force-pushed the gnuxie/eslint-9-ts-5 branch from 514b2db to 3c6cae0 Compare July 22, 2024 09:33
@Gnuxie Gnuxie merged commit 8973db4 into main Jul 22, 2024
6 checks passed
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.

1 participant