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

cli: Add ACL details to --basic-acl flag #2481

Merged
merged 2 commits into from
Aug 9, 2023

Conversation

AliceInHunterland
Copy link
Contributor

Added more info to --basic-acl flag. Closes [#2356]

@codecov
Copy link

codecov bot commented Aug 9, 2023

Codecov Report

Merging #2481 (30eef1f) into master (a7541ff) will not change coverage.
Report is 4 commits behind head on master.
The diff coverage is n/a.

❗ Current head 30eef1f differs from pull request most recent head 59a2815. Consider uploading reports for the commit 59a2815 to get more accurate results

@@           Coverage Diff           @@
##           master    #2481   +/-   ##
=======================================
  Coverage   29.24%   29.24%           
=======================================
  Files         399      399           
  Lines       30368    30368           
=======================================
  Hits         8881     8881           
  Misses      20751    20751           
  Partials      736      736           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@cthulhu-rider
Copy link
Contributor

cthulhu-rider commented Aug 9, 2023

@AliceInHunterland congrats with ur 1st PR 🌵

lets resolve DCO remark and also pay attention to @roman-khimov comment #2356 (comment)

@AliceInHunterland AliceInHunterland force-pushed the feature/2356-add-acl-details branch 2 times, most recently from f9a90bc to acab21a Compare August 9, 2023 11:52
@AliceInHunterland AliceInHunterland changed the title [#2356] Add ACL details to --basic-acl flag cli: Add ACL details to --basic-acl flag Aug 9, 2023
Closes #2356.

Signed-off-by: Ekaterina Pavlova <[email protected]>
Refs #2356.

Signed-off-by: Ekaterina Pavlova <[email protected]>
@cthulhu-rider cthulhu-rider merged commit 9f0077e into master Aug 9, 2023
6 of 8 checks passed
@cthulhu-rider cthulhu-rider deleted the feature/2356-add-acl-details branch August 9, 2023 14:55
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