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

Fix broken URL for docker-compose.yaml #26726

Merged
merged 1 commit into from Sep 27, 2022
Merged

Conversation

kaxil
Copy link
Member

@kaxil kaxil commented Sep 27, 2022

This actually fixes it. There was some attempts of it on #26721, #26695, #26711


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

.. jinja:: quick_start_ctx

To deploy Airflow on Docker Compose, you should fetch `docker-compose.yaml <{{ doc_root_url }}docker-compose.yaml>`__.
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works because it uses the same URL that is on L63 :)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

@jedcunningham jedcunningham added this to the Airflow 2.4.2 milestone Sep 27, 2022
@potiuk
Copy link
Member

potiuk commented Sep 27, 2022

4th time is a charm ?

This actually fixes it. There was some attempts of it on apache#26721, apache#26695, apache#26711
@kaxil kaxil merged commit b4ce294 into apache:main Sep 27, 2022
@kaxil kaxil deleted the fix-doc-compose branch September 27, 2022 21:42
@ephraimbuddy ephraimbuddy added the type:bug-fix Changelog: Bug Fixes label Oct 18, 2022
ephraimbuddy pushed a commit that referenced this pull request Oct 18, 2022
This actually fixes it. There was some attempts of it on #26721, #26695, #26711

(cherry picked from commit b4ce294)
ephraimbuddy pushed a commit that referenced this pull request Oct 18, 2022
This actually fixes it. There was some attempts of it on #26721, #26695, #26711

(cherry picked from commit b4ce294)
@ephraimbuddy ephraimbuddy added type:doc-only Changelog: Doc Only and removed type:bug-fix Changelog: Bug Fixes labels Oct 19, 2022
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.

None yet

5 participants