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

Notify the difference from the previous scan result #392

Merged
merged 3 commits into from
Mar 23, 2017

Conversation

knqyf263
Copy link
Contributor

What did you implement:

Closes #255

How did you implement it:

Compare differences from previous results

  • Search the old directories if result does not exist in the previous directory

How can we verify it:

$ vuls report -diff

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 c7019de into future-architect:master Mar 23, 2017
@kotakanbe kotakanbe added this to the v0.3.0 milestone Mar 24, 2017
@knqyf263 knqyf263 deleted the add_diff branch March 24, 2017 07:28
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.

3 participants