Skip to content

Deprecated vars strike through and unused var fade + codehints and lsp preferences#3038

Merged
abose merged 3 commits into
mainfrom
ai
Jul 25, 2026
Merged

Deprecated vars strike through and unused var fade + codehints and lsp preferences#3038
abose merged 3 commits into
mainfrom
ai

feat(lsp): drop untagged hint diagnostics, style unused/deprecated code

4fa5330
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Jul 25, 2026 in 48s

Annotations

Check warning on line 914 in src/extensions/default/TypeScriptSupport/unittests.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `.includes()`, rather than `.indexOf()`, when checking for existence.

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ-Yrfb2BaRu4znBW8kC&open=AZ-Yrfb2BaRu4znBW8kC&pullRequest=3038

Check warning on line 944 in src/extensions/default/TypeScriptSupport/unittests.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read.

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ-Yrfb2BaRu4znBW8kF&open=AZ-Yrfb2BaRu4znBW8kF&pullRequest=3038

Check warning on line 946 in src/languageTools/DefaultProviders.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement.

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ-Yrfh1BaRu4znBW8kI&open=AZ-Yrfh1BaRu4znBW8kI&pullRequest=3038

Check warning on line 221 in src/languageTools/LSPClient.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read.

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ-Yrfi7BaRu4znBW8kJ&open=AZ-Yrfi7BaRu4znBW8kJ&pullRequest=3038

Check warning on line 439 in src/language/CodeInspection.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `.includes()`, rather than `.indexOf()`, when checking for existence.

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ-YrfgOBaRu4znBW8kH&open=AZ-YrfgOBaRu4znBW8kH&pullRequest=3038

Check warning on line 915 in src/extensions/default/TypeScriptSupport/unittests.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `.includes()`, rather than `.indexOf()`, when checking for existence.

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ-Yrfb2BaRu4znBW8kD&open=AZ-Yrfb2BaRu4znBW8kD&pullRequest=3038

Check warning on line 436 in src/language/CodeInspection.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `.includes()`, rather than `.indexOf()`, when checking for existence.

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ-YrfgOBaRu4znBW8kG&open=AZ-YrfgOBaRu4znBW8kG&pullRequest=3038

Check warning on line 833 in src/languageTools/LSPClient.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Move function '_codeIntelPrefKey' to the outer scope.

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ-YfunfRVSYGxsIErFk&open=AZ-YfunfRVSYGxsIErFk&pullRequest=3038

Check warning on line 942 in src/extensions/default/TypeScriptSupport/unittests.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `.includes()`, rather than `.indexOf()`, when checking for existence.

See more on https://sonarcloud.io/project/issues?id=phcode-dev_phoenix&issues=AZ-Yrfb2BaRu4znBW8kE&open=AZ-Yrfb2BaRu4znBW8kE&pullRequest=3038