Skip to content

Conversation

@jscheffl
Copy link
Contributor

Another small increment to remove global statements for PR #58116

This removes some global statements in breeze for keeping track of PROVIDER_DEPENDENCIES. As of changing refactored the methods of generation and using a functools.cache() now.

global is evil. Also in breeze :-D

Not sure whether we should backport this...

@jscheffl jscheffl force-pushed the bugfix/remove-global-provider-dependencies branch from 1b90718 to 1ba05f1 Compare November 29, 2025 22:28
@potiuk potiuk merged commit 9e6c15f into apache:main Nov 30, 2025
118 checks passed
@github-actions
Copy link

Backport failed to create: v3-1-test. View the failure log Run details

Status Branch Result
v3-1-test Commit Link

You can attempt to backport this manually by running:

cherry_picker 9e6c15f v3-1-test

This should apply the commit to the v3-1-test branch and leave the commit in conflict state marking
the files that need manual conflict resolution.

After you have resolved the conflicts, you can continue the backport process by running:

cherry_picker --continue

jscheffl added a commit to jscheffl/airflow that referenced this pull request Nov 30, 2025
…8857)

* Remove global from breeze PROVIDER_DEPENDENCIES

* Consolidate constants

* Refactor provider package tooling from global constants to provider dependencies

* Add typo annotation for mypy
(cherry picked from commit 9e6c15f)

Co-authored-by: Jens Scheffler <95105677+jscheffl@users.noreply.github.com>
potiuk pushed a commit that referenced this pull request Nov 30, 2025
…58865)

* Remove global from breeze PROVIDER_DEPENDENCIES

* Consolidate constants

* Refactor provider package tooling from global constants to provider dependencies

* Add typo annotation for mypy
(cherry picked from commit 9e6c15f)
ephraimbuddy pushed a commit that referenced this pull request Dec 3, 2025
…58865)

* Remove global from breeze PROVIDER_DEPENDENCIES

* Consolidate constants

* Refactor provider package tooling from global constants to provider dependencies

* Add typo annotation for mypy
(cherry picked from commit 9e6c15f)
RoyLee1224 pushed a commit to RoyLee1224/airflow that referenced this pull request Dec 3, 2025
* Remove global from breeze PROVIDER_DEPENDENCIES

* Consolidate constants

* Refactor provider package tooling from global constants to provider dependencies

* Add typo annotation for mypy
Copilot AI pushed a commit to jason810496/airflow that referenced this pull request Dec 5, 2025
* Remove global from breeze PROVIDER_DEPENDENCIES

* Consolidate constants

* Refactor provider package tooling from global constants to provider dependencies

* Add typo annotation for mypy
itayweb pushed a commit to itayweb/airflow that referenced this pull request Dec 6, 2025
* Remove global from breeze PROVIDER_DEPENDENCIES

* Consolidate constants

* Refactor provider package tooling from global constants to provider dependencies

* Add typo annotation for mypy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants