Skip to content

Remove STATE_COLORS from Airflow Local Setting - #49228

Merged
kaxil merged 1 commit into
apache:mainfrom
astronomer:remove-statecolors
Apr 14, 2025
Merged

kaxil merged 1 commit into
apache:mainfrom
astronomer:remove-statecolors

Conversation

@kaxil

@kaxil kaxil commented Apr 14, 2025 •

Copy link
Copy Markdown
Member

part of #43519.

Since the new UI has its own color setting and we no longer allow this, remove it from here


^ 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.

@eladkal

eladkal commented Apr 14, 2025

Copy link
Copy Markdown
Contributor

do we need to mention something in change log / migration tools?
I assume that if users keep this in their local settings it will simply be ignore rather than fail deployments?

@kaxil

kaxil commented Apr 14, 2025

Copy link
Copy Markdown
Member Author

I assume that if users keep this in their local settings it will simply be ignore rather than fail deployments?

Correct

@kaxil
kaxil force-pushed the remove-statecolors branch from ea69e99 to c28db15 Compare April 14, 2025 16:03
@kaxil
kaxil force-pushed the remove-statecolors branch from c28db15 to 6135828 Compare April 14, 2025 16:47
@kaxil
kaxil force-pushed the remove-statecolors branch from 6135828 to 88a7936 Compare April 14, 2025 16:49
@kaxil
kaxil merged commit 647de21 into apache:main Apr 14, 2025
@kaxil
kaxil deleted the remove-statecolors branch April 14, 2025 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants