Skip to content

Conversation

@nicolas-grekas
Copy link
Owner

@nicolas-grekas nicolas-grekas commented Jul 22, 2025

| License | MIT

  • check executable files this is already broken on fabbot's side
  • check exception messages
  • php-cs-fixer
  • check deprecation messages
  • check json files
  • check merge in PR history
  • check for common typos
  • reject void in test cases
  • check yaml files
  • check PR header

@nicolas-grekas nicolas-grekas force-pushed the gha-fabbot branch 23 times, most recently from 7dd980b to a310614 Compare July 23, 2025 07:41
@nicolas-grekas nicolas-grekas force-pushed the gha-fabbot branch 6 times, most recently from 0552f60 to 1ab3654 Compare July 23, 2025 08:47
@nicolas-grekas nicolas-grekas force-pushed the gha-fabbot branch 2 times, most recently from 02cdd71 to 0c6a2e2 Compare July 24, 2025 16:25
@nicolas-grekas nicolas-grekas force-pushed the gha-fabbot branch 2 times, most recently from 345bdc3 to d59d5b2 Compare July 25, 2025 08:46
nicolas-grekas added a commit to symfony/symfony that referenced this pull request Jul 28, 2025
This PR was merged into the 6.4 branch.

Discussion
----------

Turn fabbot into a reusable github action

| Q             | A
| ------------- | ---
| Branch?       | 6.4
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Issues        | -
| License       | MIT

Let's move CS checks currently done by the remote app into a github action. This will remove the current bottleneck related to its maintenance, since only a few of us have access to fabbot's private repo + host.

You can see this running at nicolas-grekas#46

Commits
-------

22cec79 Turn fabbot into a reusable github action
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants