Skip to content

Fix "conftest pull" command's help message #333

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

Merged

Conversation

KeisukeYamashita
Copy link
Contributor

@KeisukeYamashita KeisukeYamashita commented Jun 27, 2020

What

I've...

  • removed extra new line&space in conftest help command
  • removed Google Cloud GCP. Added Google Cloud Storage instead.

Why

Google Cloud GCP is not exactly accurate because GCP stands for Google Cloud Platform so Google Cloud GCP will mean Google Cloud Google Cloud Platform which is not correct.

As I mentioned in #326 , GCP's GCR is one of OCI registries(ref: Container image formats, Google Cloud). Therefore, we can remove it from the help message.

Signed-off-by: KeisukeYamashita 19yamashita15@gmail.com

Signed-off-by: KeisukeYamashita <19yamashita15@gmail.com>
Signed-off-by: KeisukeYamashita <19yamashita15@gmail.com>
Signed-off-by: KeisukeYamashita <19yamashita15@gmail.com>
@jpreese
Copy link
Member

jpreese commented Jul 2, 2020

LGTM 🌮

@jpreese jpreese self-requested a review July 2, 2020 04:29
@jpreese jpreese merged commit 0ed59eb into open-policy-agent:master Jul 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants