-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Refactoring/gcp/cloudresourcemanager #231
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #231 +/- ##
===========================================
+ Coverage 35.17% 35.29% +0.12%
===========================================
Files 174 176 +2
Lines 5715 5734 +19
===========================================
+ Hits 2010 2024 +14
- Misses 3705 3710 +5
Continue to review full report at Codecov.
|
👏 👏 👏 |
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.
Same same.
…p/cloudresourcemanager
ScoutSuite/providers/gcp/resources/cloudresourcemanager/service.py
Outdated
Show resolved
Hide resolved
ScoutSuite/providers/gcp/resources/cloudresourcemanager/service.py
Outdated
Show resolved
Hide resolved
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.
LGTM 👍
…e.py Co-Authored-By: Remi05 <remi_pelletier_6@hotmail.com>
…p/cloudresourcemanager
Migrated GCP's Cloud Resource Manager service to the new architecture.
Note that
gcp/resources/projects.py
andgcp/facade/base.py
are the same as in the Stackdriver logging PR and the IAM PR; they are simply shared by all services and will appear in all GCP refactoring PRs until one of the services is merged.