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
feat: add aws ecr_repository_policy
resource
#1494
Conversation
7364723
to
5ca79c0
Compare
Codecov Report
@@ Coverage Diff @@
## main #1494 +/- ##
==========================================
- Coverage 81.71% 81.71% -0.01%
==========================================
Files 439 440 +1
Lines 16023 16059 +36
==========================================
+ Hits 13093 13122 +29
- Misses 2613 2618 +5
- Partials 317 319 +2
|
@almog27 This one seems to be blocked in review for a couple of days, is there something blocking us there ? |
771e368
to
dc392e0
Compare
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.
one of the error is incorrect but everything else LGTM. You could fix that and rebase so we can merge this pr.
Also, we should probably write a ticket to support deepmode for this one as policy support without policy content is not that useful (cc @sjourdan )
046752c
to
48b668b
Compare
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.
π
7186dd0
to
ab64949
Compare
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 π
ab64949
to
af8b0a3
Compare
af8b0a3
to
7fdc7cf
Compare
Description
This PR adds
ecr_repository_policy
resource as part of the AWS coverage