Skip to content

Turn fabbot into a reusable github action #61256

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.

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

Merged
merged 1 commit into from
Jul 28, 2025

Conversation

nicolas-grekas
Copy link
Member

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

@carsonbot carsonbot added this to the 6.4 milestone Jul 28, 2025
@nicolas-grekas nicolas-grekas force-pushed the fabbot-as-gha branch 2 times, most recently from f72521b to 5c94873 Compare July 28, 2025 10:28
@nicolas-grekas nicolas-grekas changed the title Turn fabbot into a github action Turn fabbot into a reusable github action Jul 28, 2025
@tucksaun
Copy link
Contributor

Would this make sense to extract the action in a dedicated repo (in Symfony-tools maybe?) in order to be able to reuse it more easily?

but maybe in a second iteration?

Copy link
Member

@alexandre-daubois alexandre-daubois left a comment

Choose a reason for hiding this comment

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

Really nice!

@nicolas-grekas
Copy link
Member Author

@tucksaun that's possible but any repo fits, so it's also fine here I believe: it should make it easier to tweak the action and improve it.

@nicolas-grekas nicolas-grekas merged commit 571373d into symfony:6.4 Jul 28, 2025
9 of 11 checks passed
@nicolas-grekas nicolas-grekas deleted the fabbot-as-gha branch July 28, 2025 17:04
nicolas-grekas added a commit to twigphp/Twig that referenced this pull request Jul 29, 2025
This PR was merged into the 3.x branch.

Discussion
----------

Enable Fabbot as a GHA

Follows symfony/symfony#61256

Commits
-------

a1ea8ed Enable Fabbot as a GHA
nicolas-grekas added a commit that referenced this pull request Jul 29, 2025
This PR was merged into the 6.4 branch.

Discussion
----------

Leverage symfony-tools/fabbot

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

Follows #61256
Having fabbot in another repo allows proper referencing of the workflow. This is especially important for repositories that are not symfony/symfony

Commits
-------

42a44ac Leverage symfony-tools/fabbot
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.

4 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy