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

Clarify owner links document #26515

Merged
merged 2 commits into from Sep 20, 2022
Merged

Clarify owner links document #26515

merged 2 commits into from Sep 20, 2022

Conversation

BasPH
Copy link
Contributor

@BasPH BasPH commented Sep 20, 2022

This PR fixes several code errors in the owner links doc, rewords some sentences for clarification, and adds examples for all options.

After:
image

Before:
image


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

@kaxil kaxil merged commit 14c071c into apache:main Sep 20, 2022
@BasPH BasPH deleted the reword-owner-links-doc branch September 20, 2022 18:52
@jedcunningham jedcunningham added this to the Airflow 2.4.1 milestone Sep 23, 2022
@jedcunningham jedcunningham added the type:doc-only Changelog: Doc Only label Sep 23, 2022
jedcunningham pushed a commit that referenced this pull request Sep 26, 2022
This PR fixes several code errors in the owner links doc, rewords some sentences for clarification, and adds examples for all options.

(cherry picked from commit 14c071c)
jedcunningham pushed a commit that referenced this pull request Sep 27, 2022
This PR fixes several code errors in the owner links doc, rewords some sentences for clarification, and adds examples for all options.

(cherry picked from commit 14c071c)
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

3 participants