-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
refactor(microsoft365): rename checks to match naming convention #7235
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #7235 +/- ##
==========================================
- Coverage 92.23% 88.66% -3.57%
==========================================
Files 81 1219 +1138
Lines 7356 35549 +28193
==========================================
+ Hits 6785 31521 +24736
- Misses 571 4028 +3457
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this PR only has naming changes, we need to review the tests, the coverage is lower and in principle should be the same
…te_tenants" This reverts commit 55a8310.
…o PRWLR-6455-change-microsoft-365-check-names
Context
We need to change Microsoft365 check names to make them match Prowler’s naming convention:
prowler/providers/<provider>/services/<service>/<check_name>
Checks - Prowler Open Source Documentation
Description
All check names have been updated according to Prowler’s convention.
Checklist
API
License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.