Skip to content

Remove global variables in airflow.settings#61917

Draft
jscheffl wants to merge 3 commits intoapache:mainfrom
jscheffl:bugfix/remove-global-from-settings
Draft

Remove global variables in airflow.settings#61917
jscheffl wants to merge 3 commits intoapache:mainfrom
jscheffl:bugfix/remove-global-from-settings

Conversation

@jscheffl
Copy link
Contributor

Another small increment to remove global statements for PR #58116 - an alternative approach to #61356.

This removes global statements from settings.py where the a lot of state was hold in variables used with global and then a lot of code way relying to access these globals. Is a like a public interface and refactoring would be needed.

global is evil.

Was generative AI tooling used to co-author this PR?
  • No (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@jscheffl jscheffl force-pushed the bugfix/remove-global-from-settings branch from 6aee6ee to 9925fec Compare February 15, 2026 10:29
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.

1 participant