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 for permissions batch update #612

Merged

Conversation

fmigneault
Copy link
Collaborator

@fmigneault fmigneault self-assigned this Jun 6, 2024
@github-actions github-actions bot added tests Test execution or additional use cases cli Something related to the CLI helpers labels Jun 6, 2024
@github-actions github-actions bot added the doc Documentation improvements or building problem label Jun 7, 2024
@fmigneault fmigneault marked this pull request as ready for review June 7, 2024 02:05
Copy link

codecov bot commented Jun 7, 2024

Codecov Report

Attention: Patch coverage is 83.05085% with 10 lines in your changes missing coverage. Please review.

Project coverage is 81.18%. Comparing base (5ee83c5) to head (95eef9d).

Current head 95eef9d differs from pull request most recent head 86ac29e

Please upload reports for the commit 86ac29e to get more accurate results.

Files Patch % Lines
magpie/register.py 77.41% 6 Missing and 1 partial ⚠️
magpie/cli/batch_update_permissions.py 89.28% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@                      Coverage Diff                       @@
##           fix-register-perms-request     #612      +/-   ##
==============================================================
+ Coverage                       80.88%   81.18%   +0.29%     
==============================================================
  Files                              73       74       +1     
  Lines                           10191    10240      +49     
  Branches                         1825     1835      +10     
==============================================================
+ Hits                             8243     8313      +70     
+ Misses                           1625     1604      -21     
  Partials                          323      323              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-actions github-actions bot added the ci Something related to code tests, deployment and packaging label Jun 7, 2024
@fmigneault fmigneault merged commit 5285b6e into fix-register-perms-request Jun 10, 2024
19 checks passed
@fmigneault fmigneault deleted the cli-test-register-perms-request branch June 10, 2024 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Something related to code tests, deployment and packaging cli Something related to the CLI helpers doc Documentation improvements or building problem tests Test execution or additional use cases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Provide a CLI utility to directly load a set of permissions and submit it to a remote instance
1 participant