Skip to content

Move role guide to access control#10755

Merged
mik-laj merged 2 commits intoapache:masterfrom
PolideaInternal:role-guide
Sep 6, 2020
Merged

Move role guide to access control#10755
mik-laj merged 2 commits intoapache:masterfrom
PolideaInternal:role-guide

Conversation

@mik-laj
Copy link
Member

@mik-laj mik-laj commented Sep 5, 2020

Recently, we split the security.rst page into several smaller pages, including access control. This gives us the opportunity to expand this articles with new content. Previously, adding very detailed information to the security.rst file could make it excessively long and difficult to read. For this reason, I decided to combine similar pages into one thematic article.


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, 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 UPDATING.md.

@mik-laj mik-laj changed the title Move role guides to access control Move role guide to access control Sep 5, 2020
@potiuk
Copy link
Member

potiuk commented Sep 6, 2020

Failed checks. Otherwise lgtm

Copy link
Member

@kaxil kaxil left a comment

Choose a reason for hiding this comment

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

LGTM but can you please add a PR description on why you feel that is the better place, it is easier for the community to understand why something was changes

@kaxil
Copy link
Member

kaxil commented Sep 6, 2020

Awesome, thnx

@mik-laj mik-laj merged commit 9f5b22f into apache:master Sep 6, 2020
@mik-laj mik-laj deleted the role-guide branch September 6, 2020 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants