refactor: Change gator test to gator verify
#1799
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
In this weeks community meeting, we discussed naming that would differentiate the current `gator test` from a new command that validates entire manifests against sets of policies (open-policy-agent#1786). We decided to align gator's commands with the verbs used in conftest. Accordingly, what has previously been known as `gator test` will be henceforth called: `gator verify` This PR changes package names and the name of the command. The aforementioned PR will rebase on this PR to include the name changes. Signed-off-by: juliankatz <juliankatz@google.com>
julianKatz
requested review from
shomron,
sozercan,
ritazh,
willbeason and
maxsmythe
sozercan
approved these changes
Jan 19, 2022
Codecov Report
@@ Coverage Diff @@
## master #1799 +/- ##
==========================================
- Coverage 52.08% 51.83% -0.25%
==========================================
Files 99 99
Lines 8821 8826 +5
==========================================
- Hits 4594 4575 -19
- Misses 3865 3886 +21
- Partials 362 365 +3
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
julianKatz
changed the title
gator test to gator verifygator test to gator verify
julianKatz
added a commit
to julianKatz/gatekeeper
that referenced
this pull request
Jan 19, 2022
A previous PR (open-policy-agent#1799) changed the gator test command to now be `gator verify`. That PR left out docs changes. Those docs updates are included here. Signed-off-by: juliankatz <juliankatz@google.com>
julianKatz
added a commit
that referenced
this pull request
Jan 19, 2022
* Update docs: gator test --> gator verify A previous PR (#1799) changed the gator test command to now be `gator verify`. That PR left out docs changes. Those docs updates are included here. 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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In this week's community meeting, we discussed naming that would
differentiate the current
gator testfrom a new command that validatesentire manifests against sets of policies (#1786). We decided to align
gator's commands with the verbs used in conftest.
Accordingly, what has previously been known as
gator testwill behenceforth called:
gator verifyThis PR changes package names and the name of the command. The
aforementioned PR will rebase on this PR to include the name changes.
Signed-off-by: juliankatz juliankatz@google.com