-
Notifications
You must be signed in to change notification settings - Fork 197
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
mockGCP: mock containeranalysisnote #1844
mockGCP: mock containeranalysisnote #1844
Conversation
8344460
to
1d4da87
Compare
d00f08d
to
007a0ce
Compare
fe56bbc
to
470407d
Compare
...xture/testdata/basic/containeranalysis/v1beta1/containeranalysisnote/_vcr_cassettes/dcl.yaml
Outdated
Show resolved
Hide resolved
a8ee24f
to
fc8d6bf
Compare
fc8d6bf
to
1e19291
Compare
09a7c8b
to
338be2e
Compare
mockgcp/bigquery-v2.json
Outdated
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.
Can we group these json file and put them under mockgcp/mockcontaineranalysis
?
...est/resourcefixture/testdata/basic/containeranalysis/v1beta1/containeranalysisnote/_http.log
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.
nit: Shall we put these under third_party/googlepais/mockcontaineranalysisnote/ ? What I"m thinking is for future maintenance, under mockgrafeas could make it harder to discover and reuse. But this is just a nit
/lgtm |
/hold for addressing comments |
338be2e
to
ba56aab
Compare
ba56aab
to
7257514
Compare
7257514
to
5ad80ee
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
/approve
Thanks!
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: yuwenma The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
0f65a10
into
GoogleCloudPlatform:master
Change description
Add mockGCP for ContainerAnalysisNote
Tests you have done
make ready-pr
to ensure this PR is ready for review.