Skip to content
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

Add an option to mount airflow-site/sphinx_airflow_theme when running CI locally #45576

Open
1 of 2 tasks
shahar1 opened this issue Jan 11, 2025 · 1 comment
Open
1 of 2 tasks
Labels
area:CI Airflow's tests and continious integration area:dev-env-Breeze2 Breeze 2 issues kind:feature Feature Requests

Comments

@shahar1
Copy link
Contributor

shahar1 commented Jan 11, 2025

Description

As of today, making changes in Airflow's site theme requires a lot of effort, as sphinx_airflow_theme is installed from PyPi:

"sphinx-airflow-theme>=0.0.12",

It doesn't happen often, but when we need to modify it for any reason (for example, as part of upgrading Sphinx version and related components - see this PR), we need to make a lot of workarounds to make it work.
Having the option to mount local airflow-site directory will make it way easier to test.

Use case/motivation

No response

Related issues

No response

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@shahar1 shahar1 added kind:feature Feature Requests area:dev-env-Breeze2 Breeze 2 issues labels Jan 11, 2025
@dosubot dosubot bot added the area:CI Airflow's tests and continious integration label Jan 11, 2025
@potiuk
Copy link
Member

potiuk commented Jan 11, 2025

As discussed on slack. One of the solutions is to add it our uv workspace in airflow and build it from there. I might want to take a stab with that one soon-ish.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:CI Airflow's tests and continious integration area:dev-env-Breeze2 Breeze 2 issues kind:feature Feature Requests
Projects
None yet
Development

No branches or pull requests

2 participants