coveralls
commented
5 months ago Pull Request Test Coverage Report for Build 8578510842
Details
- 4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
- No unchanged relevant lines lost coverage.
- Overall coverage decreased (-0.02%) to 50.122%
| Totals | |
|---|---|
| Change from base Build 8391281769: | -0.02% |
| Covered Lines: | 1236 |
| Relevant Lines: | 2466 |
Context
Summary
This PR can be summarized in the following changelog entry:
Relevant technical choices:
Composer: update dev dependencies
Nearly all dev dependencies have had new releases. This commit updates the plugin to use the new versions.
For linting, the upgrade will get us preliminary PHP 8.4 support. For CS, the upgrades will get us improved syntax support for PHP 8.3, more documentation and a range of bug fixes.
CS/check thresholds: require exact match for thresholds
This changes the implementation of the coding standards threshold check to require that both the
YOASTCS_THRESHOLD_ERRORSenvironment variable, as well as theYOASTCS_THRESHOLD_WARNINGSenvironment variable match the current status exactly.This prevents PR A fixing some issues and forgetting to update the threshold, which then would allow PR B to introduce new issues.
CS: minor clean up
Test instructions
Test instructions for the acceptance test before the PR gets merged
This PR can be acceptance tested by following these steps: