Skip to content

chore: consider switching from py to .sh +... #4499

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.

Sign up for GitHub

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

Open
nickboldt wants to merge 5 commits into
base: main
Choose a base branch
from

add yarn lock for install-dynamic-plugins project/script

82678c1
Select commit
Loading
Failed to load commit list.
Open

chore: consider switching from py to .sh +... #4499

add yarn lock for install-dynamic-plugins project/script
82678c1
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Mar 31, 2026 in 46s

Quality Gate failed

Failed conditions
14 Security Hotspots
D Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Annotations

Check warning on line 67 in scripts/install-dynamic-plugins/src/registry-oci.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `String#replaceAll()` over `String#replace()`. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1lz7c7kP5Fazkn3&open=AZ1AW1lz7c7kP5Fazkn3&pullRequest=4499

Check warning on line 48 in scripts/install-dynamic-plugins/src/npm-parse-plugin-key.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This assertion is unnecessary since it does not change the type of the expression. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1f07c7kP5Fazkm_&open=AZ1AW1f07c7kP5Fazkm_&pullRequest=4499

Check warning on line 164 in scripts/install-dynamic-plugins/src/merge-dynamic-plugins.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This assertion is unnecessary since it does not change the type of the expression. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1mF7c7kP5FazkoJ&open=AZ1AW1mF7c7kP5FazkoJ&pullRequest=4499

Check warning on line 47 in scripts/install-dynamic-plugins/src/registry-oci.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use the "RegExp.exec()" method instead. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1lz7c7kP5Fazkn2&open=AZ1AW1lz7c7kP5Fazkn2&pullRequest=4499

Check warning on line 13 in scripts/install-dynamic-plugins/src/merge-app-config.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Unexpected negated condition. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1l_7c7kP5Fazkn5&open=AZ1AW1l_7c7kP5Fazkn5&pullRequest=4499

Check warning on line 129 in scripts/install-dynamic-plugins/src/merge-dynamic-plugins.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This assertion is unnecessary since it does not change the type of the expression. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1mF7c7kP5FazkoD&open=AZ1AW1mF7c7kP5FazkoD&pullRequest=4499

Check warning on line 164 in scripts/install-dynamic-plugins/src/install.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This assertion is unnecessary since it does not change the type of the expression. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1lc7c7kP5FazknN&open=AZ1AW1lc7c7kP5FazknN&pullRequest=4499

Check warning on line 156 in scripts/install-dynamic-plugins/src/merge-dynamic-plugins.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This assertion is unnecessary since it does not change the type of the expression. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1mF7c7kP5FazkoG&open=AZ1AW1mF7c7kP5FazkoG&pullRequest=4499

Check warning on line 13 in scripts/install-dynamic-plugins/src/oci-ref.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

`String.raw` should be used to avoid escaping `\`. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1ln7c7kP5Fazkns&open=AZ1AW1ln7c7kP5Fazkns&pullRequest=4499

Check warning on line 31 in scripts/install-dynamic-plugins/src/oci-ref.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use the "RegExp.exec()" method instead. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1ln7c7kP5Fazknv&open=AZ1AW1ln7c7kP5Fazknv&pullRequest=4499

Check warning on line 8 in scripts/install-dynamic-plugins/src/oci-parse.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

`String.raw` should be used to avoid escaping `\`. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1li7c7kP5Fazknf&open=AZ1AW1li7c7kP5Fazknf&pullRequest=4499

Check warning on line 9 in scripts/install-dynamic-plugins/src/oci-parse.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

`String.raw` should be used to avoid escaping `\`. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1li7c7kP5Fazkng&open=AZ1AW1li7c7kP5Fazkng&pullRequest=4499

Check warning on line 33 in scripts/install-dynamic-plugins/src/oci-parse.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use the "RegExp.exec()" method instead. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1li7c7kP5Fazknl&open=AZ1AW1li7c7kP5Fazknl&pullRequest=4499

Check warning on line 8 in scripts/install-dynamic-plugins/src/oci-ref.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

`String.raw` should be used to avoid escaping `\`. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1ln7c7kP5Fazknp&open=AZ1AW1ln7c7kP5Fazknp&pullRequest=4499

Check warning on line 307 in scripts/install-dynamic-plugins/src/install.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

'pluginJson.forceDownload ?? false' will use Object's default stringification format ('[object Object]') when stringified. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1lc7c7kP5FazknV&open=AZ1AW1lc7c7kP5FazknV&pullRequest=4499

Check warning on line 26 in scripts/install-dynamic-plugins/src/cli.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer top-level await over using a promise chain. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1lO7c7kP5FazknB&open=AZ1AW1lO7c7kP5FazknB&pullRequest=4499

Check warning on line 17 in scripts/install-dynamic-plugins/src/oci-ref.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

`String.raw` should be used to avoid escaping `\`. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1ln7c7kP5Fazknu&open=AZ1AW1ln7c7kP5Fazknu&pullRequest=4499

Check warning on line 35 in scripts/install-dynamic-plugins/src/oci-ref.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This assertion is unnecessary since it does not change the type of the expression. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1ln7c7kP5Fazknw&open=AZ1AW1ln7c7kP5Fazknw&pullRequest=4499

Check warning on line 66 in scripts/install-dynamic-plugins/src/merge-dynamic-plugins.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This assertion is unnecessary since it does not change the type of the expression. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1mF7c7kP5Fazkn_&open=AZ1AW1mF7c7kP5Fazkn_&pullRequest=4499

Check warning on line 132 in scripts/install-dynamic-plugins/src/merge-dynamic-plugins.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This assertion is unnecessary since it does not change the type of the expression. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1mF7c7kP5FazkoE&open=AZ1AW1mF7c7kP5FazkoE&pullRequest=4499

Check warning on line 52 in scripts/install-dynamic-plugins/src/install.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `.find(…)` over `.filter(…)`. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1lc7c7kP5FazknG&open=AZ1AW1lc7c7kP5FazknG&pullRequest=4499

Check warning on line 166 in scripts/install-dynamic-plugins/src/merge-dynamic-plugins.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This assertion is unnecessary since it does not change the type of the expression. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1mF7c7kP5FazkoK&open=AZ1AW1mF7c7kP5FazkoK&pullRequest=4499

Check warning on line 81 in scripts/install-dynamic-plugins/src/install.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `String#replaceAll()` over `String#replace()`. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1lc7c7kP5FazknL&open=AZ1AW1lc7c7kP5FazknL&pullRequest=4499

Check warning on line 10 in scripts/install-dynamic-plugins/src/oci-ref.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

`String.raw` should be used to avoid escaping `\`. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1ln7c7kP5Fazknr&open=AZ1AW1ln7c7kP5Fazknr&pullRequest=4499

Check warning on line 16 in scripts/install-dynamic-plugins/src/compute-plugin-hash.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

`new Error()` is too unspecific for a type check. Use `new TypeError()` instead. 

See more on https://sonarcloud.io/project/issues?id=redhat-developer_rhdh&issues=AZ1AW1lt7c7kP5Fazkny&open=AZ1AW1lt7c7kP5Fazkny&pullRequest=4499
View more details on SonarQubeCloud