Skip to content

Add PR visual comment workflow with render tooling#1248

Open
f3l1x wants to merge 1 commit into
masterfrom
fx/visual-testing
Open

Add PR visual comment workflow with render tooling#1248
f3l1x wants to merge 1 commit into
masterfrom
fx/visual-testing

Add PR visual datagrid comment

3db348d
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis failed Mar 9, 2026 in 32s

Quality Gate failed

Failed conditions
1 Security Hotspot
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 8 in tests/visual/test.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=contributte_datagrid&issues=AZzSlvmLuIJseWZVhT7Q&open=AZzSlvmLuIJseWZVhT7Q&pullRequest=1248

Check warning on line 0 in tests/visual/test.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace all tab characters in this file by sequences of white-spaces. 

See more on https://sonarcloud.io/project/issues?id=contributte_datagrid&issues=AZzSlvmLuIJseWZVhT7O&open=AZzSlvmLuIJseWZVhT7O&pullRequest=1248

Check warning on line 3 in tests/visual/render_datagrid.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=contributte_datagrid&issues=AZzSlvl7uIJseWZVhT7L&open=AZzSlvl7uIJseWZVhT7L&pullRequest=1248

Check warning on line 42 in tests/visual/datagrid.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define and throw a dedicated exception instead of using a generic one. 

See more on https://sonarcloud.io/project/issues?id=contributte_datagrid&issues=AZzSlvmDuIJseWZVhT7N&open=AZzSlvmDuIJseWZVhT7N&pullRequest=1248

Check warning on line 7 in tests/visual/test.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=contributte_datagrid&issues=AZzSlvmLuIJseWZVhT7P&open=AZzSlvmLuIJseWZVhT7P&pullRequest=1248

Check warning on line 73 in tests/visual/test.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor the code to avoid updating the loop counter "$index" within the loop body. 

See more on https://sonarcloud.io/project/issues?id=contributte_datagrid&issues=AZzSlvmLuIJseWZVhT7R&open=AZzSlvmLuIJseWZVhT7R&pullRequest=1248

Check warning on line 0 in tests/visual/datagrid.php

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace all tab characters in this file by sequences of white-spaces. 

See more on https://sonarcloud.io/project/issues?id=contributte_datagrid&issues=AZzSlvmDuIJseWZVhT7M&open=AZzSlvmDuIJseWZVhT7M&pullRequest=1248
View more details on SonarQubeCloud