Skip to content

Auto-close PRs on subtree-splits #54653

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
Apr 18, 2024
Merged

Conversation

nicolas-grekas
Copy link
Member

@nicolas-grekas nicolas-grekas commented Apr 18, 2024

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

From time to time, people submit PRs on subtree splits.
We used to auto-close them using a bot. But maintaining a bot is boring.
Later on GitHub introduced org-wide workflows. Then removed them.
Here, I propose we add a .github folder to all our packages, with an action to auto-close PRs.

Copy link
Member

@lyrixx lyrixx left a comment

Choose a reason for hiding this comment

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

Thanks 👍🏼

@nicolas-grekas nicolas-grekas merged commit 2c9352d into symfony:5.4 Apr 18, 2024
10 of 13 checks passed
@derrabus
Copy link
Member

Awesome, thank you!

@nicolas-grekas nicolas-grekas deleted the gitplit branch April 18, 2024 09:35
kbond added a commit to symfony/ux that referenced this pull request Apr 18, 2024
This PR was merged into the 2.x branch.

Discussion
----------

Auto-close PRs on subtree-splits

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| Issues        | n/a
| License       | MIT

Same as symfony/symfony#54653

Commits
-------

63ee860 Auto-close PRs on subtree-splits
nicolas-grekas added a commit to symfony/polyfill that referenced this pull request Apr 19, 2024
This PR was merged into the 1.x branch.

Discussion
----------

Auto-close PRs on subtree-splits

Same as symfony/symfony#54653

Commits
-------

78bbd28 Auto-close PRs on subtree-splits
nicolas-grekas added a commit to twigphp/Twig that referenced this pull request Apr 19, 2024
This PR was merged into the 3.x branch.

Discussion
----------

Auto-close PRs on subtree-splits

Same as symfony/symfony#54653

Commits
-------

b212f1b Auto-close PRs on subtree-splits
fabpot added a commit that referenced this pull request Apr 26, 2024
This PR was merged into the 6.4 branch.

Discussion
----------

[Security] Remove workflow from empty folder

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

The workflows were add in #54653, but the Guard component was removed in 6.0, so there's just an empty folder now on higher versions.

Commits
-------

2aaec67 [Security] Remove workflow from empty folder
fabpot added a commit to fabpot/symfony that referenced this pull request May 31, 2024
…-grekas)"

This reverts commit 2c9352d, reversing
changes made to 18c3e87.
fabpot added a commit that referenced this pull request May 31, 2024
This PR was merged into the 5.4 branch.

Discussion
----------

Remove subtree split checks

Reverts #54653 as it does not work.

Commits
-------

ca6487e Revert "minor #54653 Auto-close PRs on subtree-splits (nicolas-grekas)"
fabpot added a commit that referenced this pull request May 31, 2024
* 5.4:
  Revert "minor #54653 Auto-close PRs on subtree-splits (nicolas-grekas)"
fabpot added a commit that referenced this pull request May 31, 2024
* 6.4:
  Remove subtree split checks
  Revert "minor #54653 Auto-close PRs on subtree-splits (nicolas-grekas)"
fabpot added a commit that referenced this pull request May 31, 2024
* 7.0:
  Remove subtree split checks
  Revert "minor #54653 Auto-close PRs on subtree-splits (nicolas-grekas)"
fabpot added a commit that referenced this pull request May 31, 2024
* 7.1:
  Remove subtree split checks
  Remove subtree split checks
  Revert "minor #54653 Auto-close PRs on subtree-splits (nicolas-grekas)"
  [SecurityBundle] Fix `container.build_hash` parameter binding
  [Serializer] Fix denormalizing a collection of union types
  [DoctrineBridge] Fix `UniqueEntityValidator` with proxy object
MindfulPol pushed a commit to MindfulPol/symfony that referenced this pull request Jun 1, 2024
…-grekas)"

This reverts commit 2c9352d, reversing
changes made to 18c3e87.
nicolas-grekas added a commit that referenced this pull request Sep 25, 2024
…itories (GromNaN)

This PR was squashed before being merged into the 5.4 branch.

Discussion
----------

Add PR template and auto-close PR on subtree split repositories

| Q             | A
| ------------- | ---
| Branch?       | 5.4
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Issues        | Fix #57267
| License       | MIT

Extract the Pull Request Template from #54653.

~The GitHub action did not work as said by `@nicolas`-grekas #57267 (comment) Replaced by https://github.com/superbrothers/close-pull-request.

Commits
-------

d90fa7a Add PR template and auto-close PR on subtree split repositories
symfonyaml pushed a commit to symfonyaml/symfony that referenced this pull request Oct 21, 2024
…-grekas)"

This reverts commit 2c9352d, reversing
changes made to 18c3e87.
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.

5 participants