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

Small changes on "DAGs and Tasks documentation" #14853

Merged
merged 6 commits into from May 9, 2021

Conversation

JavierLopezT
Copy link
Contributor

No description provided.

@JavierLopezT
Copy link
Contributor Author

I have "broken" the tutorial dag file. It was made for being one actual dag split into different sections. However, I find it confusing because in each section there was just one part of the DAG, and in some cases, as the modified, I think it is useful to have the whole picture. So before fixing the static cheks, I would like some feedback on this, just in case I have to make other changes.

Also, I don't know if with the code the SEO can be improved, because if you search DAG documentation or similar in Google it doesn't appear. It doesn't even appear in the search of Airflow docs.

@github-actions
Copy link

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@github-actions
Copy link

github-actions bot commented Apr 1, 2021

The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*.

@JavierLopezT JavierLopezT requested a review from kaxil April 27, 2021 19:22
@KulykDmytro
Copy link
Contributor

KulykDmytro commented Apr 28, 2021

task.doc_md not rendering when

AIRFLOW__CORE__STORE_SERIALIZED_DAGS=True

at least on 1.10.15

@JavierLopezT JavierLopezT changed the title Improvements on "DAGs and Tasks documentation" Little changes on "DAGs and Tasks documentation" May 8, 2021
@JavierLopezT JavierLopezT changed the title Little changes on "DAGs and Tasks documentation" Small changes on "DAGs and Tasks documentation" May 9, 2021
@potiuk potiuk merged commit 00336ad into apache:master May 9, 2021
@JavierLopezT JavierLopezT deleted the dag_docs_doc branch August 31, 2021 07:22
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.

None yet

6 participants