-
Notifications
You must be signed in to change notification settings - Fork 16.3k
Improve upgrade important version hook: #54606
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
Conversation
* we do not have to check Python versions below 3.9 - that significantly speeds up the check * we run `uv sync --resolution highest` in dev/breeze when there are some changes - that will make sure that the latest version of dependencies in breeze are captured in the uv.lock file
78b3e42 to
59f8050
Compare
gopidesupavan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice update :)
Yeah. I noticed breeze lock was out of sync after the last change and finally I found a way how we can also make it "eventually consistent". Thanks to that I have an idea how we can add uv.lock to Airflow and use the .lock instead of constraints for PRs and regular developments (we'd still generate constraints the same way we do now, but we could switch to uv.lock. Let's see how it works - but I think this |
* we do not have to check Python versions below 3.9 - that significantly speeds up the check * we run `uv sync --resolution highest` in dev/breeze when there are some changes - that will make sure that the latest version of dependencies in breeze are captured in the uv.lock file (cherry picked from commit c6c6d75) Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
* we do not have to check Python versions below 3.9 - that significantly speeds up the check * we run `uv sync --resolution highest` in dev/breeze when there are some changes - that will make sure that the latest version of dependencies in breeze are captured in the uv.lock file (cherry picked from commit c6c6d75) Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
* we do not have to check Python versions below 3.9 - that significantly speeds up the check * we run `uv sync --resolution highest` in dev/breeze when there are some changes - that will make sure that the latest version of dependencies in breeze are captured in the uv.lock file (cherry picked from commit c6c6d75) Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
* we do not have to check Python versions below 3.9 - that significantly speeds up the check * we run `uv sync --resolution highest` in dev/breeze when there are some changes - that will make sure that the latest version of dependencies in breeze are captured in the uv.lock file (cherry picked from commit c6c6d75) Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
we do not have to check Python versions below 3.9 - that significantly speeds up the check
we run
uv sync --resolution highestin dev/breeze when there are some changes - that will make sure that the latest version of dependencies in breeze are captured in the uv.lock file^ 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.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.