Skip to content

[Composer] Added one command to run them all (checks)#3271

Merged
mnocon merged 9 commits into
5.0from
one-command-to-run-them-all
Jul 15, 2026
Merged

[Composer] Added one command to run them all (checks)#3271
mnocon merged 9 commits into
5.0from
one-command-to-run-them-all

Conversation

@mnocon

@mnocon mnocon commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Target: 5.0, 6.0

This PR adds a single command to run all code and content quality checks - except Vale, which is not a dependency and cannot be run as easily.

Goal: I want to use that locally, to run all checks quickly with composer test.

In addition, some helper commands are added to regenerate baselines in a consistent way.

Usage:

  • composer test before sending the PR to review
  • composer fix in case something is detected
  • composer baseline when a baseline needs to be generated

They run composer update already, so there's less friction.

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

Preview of modified files

Preview of modified Markdown:

@mnocon mnocon force-pushed the one-command-to-run-them-all branch from affb5d3 to 948cc05 Compare July 8, 2026 08:06
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

Summary

Status Count
🔍 Total 761791
🔗 Unique 15026
✅ Successful 6425
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 755364
❓ Unknown 0
🚫 Errors 2
⛔ Unsupported 0

Errors per input

Errors in site/administration/project_organization/bundles/index.html

Errors in site/templating/twig_function_reference/recommendations_twig_functions/index.html

Full Github Actions output

@mnocon mnocon marked this pull request as ready for review July 8, 2026 08:11
@mnocon mnocon requested a review from a team July 8, 2026 08:11
@ibexa-workflow-automation-1 ibexa-workflow-automation-1 Bot requested review from adriendupuis, dabrt and julitafalcondusza and removed request for a team July 8, 2026 08:11
@mnocon mnocon force-pushed the one-command-to-run-them-all branch from d305113 to 9282c4d Compare July 15, 2026 11:16
@mnocon mnocon requested review from adriendupuis July 15, 2026 11:20

@adriendupuis adriendupuis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We lost the verbosity of which failed (and eventually, which is logging)

Comment thread composer.json Outdated
@mnocon mnocon force-pushed the one-command-to-run-them-all branch from 041ef28 to e993497 Compare July 15, 2026 14:20
@mnocon mnocon merged commit 215592a into 5.0 Jul 15, 2026
10 of 11 checks passed
@mnocon mnocon deleted the one-command-to-run-them-all branch July 15, 2026 14:28
mnocon added a commit that referenced this pull request Jul 15, 2026
* [Composer] Added one command to run them all (checks)

* Added YAML check

* Added regemerate baseline commands

* Added "fix" and "baseline" commands

* Run PHP-CS-Fixer on PHP 8.4

* Update composer.json

Co-authored-by: Adrien Dupuis <61695653+adriendupuis@users.noreply.github.com>

* Fix syntax error

* Fixed CS

* Add colors

---------

Co-authored-by: Adrien Dupuis <61695653+adriendupuis@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants