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

Create gator root command #1403

Merged
merged 3 commits into from Jun 29, 2021
Merged

Conversation

willbeason
Copy link
Member

@willbeason willbeason commented Jun 25, 2021

Per meeting, the Gatekeeper CLI root command is named "gator", and we will add several subcommands. The current one is named "test".

The changes for this PR are on top of willbeason/files. Only changes in the commit named "Make gator root command" need to be reviewed - this PR will not be merged until #1398 is merged.

Signed-off-by: Will Beason <willbeason@google.com>
Copy link
Contributor

@maxsmythe maxsmythe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, pending sozercan's nit

Per decision in weekly meeting, make the "gator" root command and "test"
the subcommand in charge of running the tests. This frees us up to add
other subcommands that we want to do later.

Signed-off-by: Will Beason <willbeason@google.com>
@willbeason willbeason requested a review from sozercan June 28, 2021 17:12
@willbeason willbeason added the authorship-tooling Tools for authors of Gatekeeper Constraints, Templates, Mutators label Jun 28, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #1403 (6a49b58) into master (60c9781) will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1403      +/-   ##
==========================================
- Coverage   49.65%   49.63%   -0.03%     
==========================================
  Files          68       68              
  Lines        4926     4926              
==========================================
- Hits         2446     2445       -1     
- Misses       2136     2137       +1     
  Partials      344      344              
Flag Coverage Δ
unittests 49.63% <ø> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...onstrainttemplate/constrainttemplate_controller.go 57.28% <0.00%> (-0.98%) ⬇️
pkg/readiness/list.go 91.30% <0.00%> (+8.69%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 60c9781...6a49b58. Read the comment docs.

@sozercan sozercan merged commit c43fb49 into open-policy-agent:master Jun 29, 2021
@willbeason willbeason deleted the root-cmd branch June 29, 2021 16:12
julianKatz pushed a commit to julianKatz/gatekeeper that referenced this pull request Jul 8, 2021
* read test files

Signed-off-by: Will Beason <willbeason@google.com>

* Make gator root command

Per decision in weekly meeting, make the "gator" root command and "test"
the subcommand in charge of running the tests. This frees us up to add
other subcommands that we want to do later.

Signed-off-by: Will Beason <willbeason@google.com>

Co-authored-by: Sertaç Özercan <852750+sozercan@users.noreply.github.com>
Signed-off-by: juliankatz <juliankatz@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
authorship-tooling Tools for authors of Gatekeeper Constraints, Templates, Mutators
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants