Skip to content
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

Add draft config option to Create Pull Request #151

Merged
merged 1 commit into from
Jan 15, 2025

Conversation

gopidesupavan
Copy link
Contributor

Adding draft parameter option to create pull request.
https://docs.github.com/en/rest/pulls/pulls?apiVersion=2022-11-28#create-a-pull-request

At Airflow, we effectively use the cherry-picker tool to backport changes, which has been highly beneficial in saving maintainers' time.

Currently, we have configured cherry-picker in our workflows, utilizing the GITHUB_TOKEN to create backport PRs. However, our workflows are not triggered on these backported PRs. This limitation occurs because GitHub prevents recursive workflows when using the default token.

https://docs.github.com/en/actions/writing-workflows/choosing-when-your-workflow-runs/triggering-a-workflow#triggering-a-workflow-from-a-workflow .

To address this, we would like to create backport PRs in draft state. maintainer can then change them to the "Ready for Review" state, which will subsequently trigger the workflows.

Thank you @potiuk for the suggestion this.

Copy link

cpython-cla-bot bot commented Dec 15, 2024

All commit authors signed the Contributor License Agreement.
CLA signed

@ambv ambv merged commit 962ba5c into python:main Jan 15, 2025
26 checks passed
@potiuk
Copy link
Contributor

potiuk commented Jan 15, 2025

Cool. Thanks @ambv !

@gopidesupavan
Copy link
Contributor Author

Oh nice , thank you @ambv :)

@gopidesupavan gopidesupavan deleted the add-pr-draft-config branch January 18, 2025 12:44
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.

3 participants