Skip to content

Conversation

@vdiez
Copy link
Contributor

@vdiez vdiez commented Nov 17, 2025

@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod bot changed the title Fix S1541: Support nullish coalescing operator JS-956 Fix S1541: Support nullish coalescing operator Nov 17, 2025
@hashicorp-vault-sonar-prod
Copy link

hashicorp-vault-sonar-prod bot commented Nov 17, 2025

JS-956

@vdiez vdiez requested a review from zglicz November 17, 2025 12:35
@sonarqube-next
Copy link

@vdiez vdiez changed the title JS-956 Fix S1541: Support nullish coalescing operator JS-955 Fix S1541: Support nullish coalescing operator Nov 17, 2025
Copy link
Contributor

@zglicz zglicz left a comment

Choose a reason for hiding this comment

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

Is this based on a community report? please reference if so.


const baseLanguageOptions: Linter.LanguageOptions = {
ecmaVersion: 2018,
ecmaVersion: 2022,
Copy link
Contributor

Choose a reason for hiding this comment

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

couldn't this be even newer?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I prefer to keep the minimum required, we can adapt in some tests contain newer syntax

@vdiez vdiez merged commit 30d37f6 into master Nov 19, 2025
40 checks passed
@vdiez vdiez deleted the JS-955 branch November 19, 2025 08: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