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

Add containers-only option to configtest #411

Conversation

knqyf263
Copy link
Contributor

What did you implement:

I'd like to scan only containers.
scan subcommand has -containers-only option.
#190

However, configtest doesn't have -containers-only option.
Then, an error about a host server occur.

How did you implement it:

I added -containers-only option to configtest.

How can we verify it:

$ vuls configtest -containers-only

Todos:

You don't have to satisfy all of the following.

  • Write tests
  • Write documentation
  • Check that there aren't other open pull requests for the same issue/feature
  • Format your source code by make fmt
  • Pass the test by make test
  • Provide verification config / commands
  • Enable "Allow edits from maintainers" for this PR
  • Update the messages below

Is this ready for review?: YES
Is it a breaking change?: NO

@kotakanbe kotakanbe merged commit 33b2aa2 into future-architect:master Apr 4, 2017
@kotakanbe
Copy link
Member

Many Thanks :)

@kotakanbe kotakanbe modified the milestone: v0.4.0 Apr 4, 2017
lapthorn pushed a commit to lapthorn/vuls that referenced this pull request May 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants