Skip to content

Detecting and fixing incorrect YAML samples #3165

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
mnocon wants to merge 16 commits into
base: 5.0
Choose a base branch
from

Speed up code samples workflow

19ca290
Select commit
Loading
Failed to load commit list.
Open

Detecting and fixing incorrect YAML samples #3165

Speed up code samples workflow
19ca290
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed May 16, 2026 in 3m 27s

Quality Gate failed

Failed conditions
E Security Rating on New Code (required ≥ A)
B 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 69 in tests/ConfigurationProvider.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This method has 4 returns, which is more than the 3 allowed. 

See more on https://sonarcloud.io/project/issues?id=ezsystems_developer-documentation&issues=AZ2558wnNaEFFWxaz5ph&open=AZ2558wnNaEFFWxaz5ph&pullRequest=3165

Check failure on line 90 in .github/workflows/code_samples.yaml

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

The expression github.head_ref can be set by an external actor to a specially crafted value, enabling script injection. Change this workflow to not use user-controlled data directly in a run block, for example by assigning this expression to an environment variable. 

See more on https://sonarcloud.io/project/issues?id=ezsystems_developer-documentation&issues=AZ4wLRtMmJyaFL6eZP3s&open=AZ4wLRtMmJyaFL6eZP3s&pullRequest=3165

Check warning on line 12 in tests/bootstrap.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace "require" with "require_once". 

See more on https://sonarcloud.io/project/issues?id=ezsystems_developer-documentation&issues=AZ2558tUNaEFFWxaz5pg&open=AZ2558tUNaEFFWxaz5pg&pullRequest=3165

Check warning on line 28 in tests/ValidationBaseline.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused "$repoRoot" private field. 

See more on https://sonarcloud.io/project/issues?id=ezsystems_developer-documentation&issues=AZ2558wwNaEFFWxaz5pj&open=AZ2558wwNaEFFWxaz5pj&pullRequest=3165

Check warning on line 64 in tests/ConfigurationProvider.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Immediately return this expression instead of assigning it to the temporary variable "$result". 

See more on https://sonarcloud.io/project/issues?id=ezsystems_developer-documentation&issues=AZ2558wnNaEFFWxaz5pi&open=AZ2558wnNaEFFWxaz5pi&pullRequest=3165

Check warning on line 91 in tests/generate-yaml-baseline.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

This branch's code block is the same as the block for the branch on line 78. 

See more on https://sonarcloud.io/project/issues?id=ezsystems_developer-documentation&issues=AZ4wFNcEmJyaFL6eYY8p&open=AZ4wFNcEmJyaFL6eYY8p&pullRequest=3165
View more details on SonarQubeCloud