Skip to content

[v3-1-test] Remove duplicate cooldown-days flag in basic-tests.yml#59629

Merged
amoghrajesh merged 1 commit intoapache:v3-1-testfrom
amoghrajesh:remove-duplicate-cooldown-days
Dec 19, 2025
Merged

[v3-1-test] Remove duplicate cooldown-days flag in basic-tests.yml#59629
amoghrajesh merged 1 commit intoapache:v3-1-testfrom
amoghrajesh:remove-duplicate-cooldown-days

Conversation

@amoghrajesh
Copy link
Copy Markdown
Contributor

https://github.com/apache/airflow/pull/59614/changes accidentally added duplicate flag leading to v3 branch being broken with:

Run prek auto-update --cooldown-days 4 --cooldown-days 4 --freeze
error: the argument '--cooldown-days <DAYS>' cannot be used multiple times

Usage: prek auto-update [OPTIONS]

For more information, try '--help'.
Error: Process completed with exit code 2.

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@amoghrajesh
Copy link
Copy Markdown
Contributor Author

Failure is unrelated. Fixed separately

@amoghrajesh amoghrajesh merged commit 65ac8d5 into apache:v3-1-test Dec 19, 2025
86 of 87 checks passed
@potiuk
Copy link
Copy Markdown
Member

potiuk commented Dec 19, 2025

🤦 ... That was a cool change :)

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.

6 participants