Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

A bunch of text edits #130

Merged
merged 6 commits into from
May 15, 2018
Merged

A bunch of text edits #130

merged 6 commits into from
May 15, 2018

Conversation

ttousai
Copy link
Contributor

@ttousai ttousai commented May 15, 2018

No description provided.

ttousai added 5 commits May 15, 2018 04:08
it has grown stale and is dependent on k8s version we are checking.
It has grown stale and no longer reflects the supported options, and can be misleading (see #127).
@ttousai ttousai requested a review from lizrice May 15, 2018 04:45
Copy link
Contributor

@lizrice lizrice left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@@ -46,7 +46,7 @@ var (
var RootCmd = &cobra.Command{
Use: os.Args[0],
Short: "Run CIS Benchmarks checks against a Kubernetes deployment",
Long: `This tool runs the CIS Kubernetes 1.6 Benchmark v1.0.0 checks.`,
Long: `This tool runs the CIS Kubernetes Benchmark (http://www.cisecurity.org/benchmark/kubernetes/)`,
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Good idea to make this more future-proof!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants