Skip to content

JS-594 Remove eslint scope#5151

Merged
vdiez merged 2 commits intomasterfrom
remove-eslint-scope
Mar 3, 2025
Merged

JS-594 Remove eslint scope#5151
vdiez merged 2 commits intomasterfrom
remove-eslint-scope

Conversation

@zglicz
Copy link
Copy Markdown
Contributor

@zglicz zglicz commented Mar 2, 2025

JS-594

Digging into why we have an outdated version of @types/[email protected], it is because the types of @types/eslint-scope requires @types/eslint which is a deprecated package - DefinitelyTyped/DefinitelyTyped#70735 .

eslint package now publishes their own types and by removing the dependency on eslint-scope, we get perfectly typed eslint package.

@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod bot changed the title Remove eslint scope JS-594 Remove eslint scope Mar 2, 2025
@sonarqube-next
Copy link
Copy Markdown

sonarqube-next bot commented Mar 2, 2025

Copy link
Copy Markdown
Contributor

@vdiez vdiez left a comment

Choose a reason for hiding this comment

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

Nice catch!

@vdiez vdiez merged commit 8c30a6b into master Mar 3, 2025
19 of 20 checks passed
@vdiez vdiez deleted the remove-eslint-scope branch March 3, 2025 09:32
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.

2 participants