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.

Sign up for GitHub

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

Jump to bottom

Add more validations #329

Merged
merged 2 commits into from
Feb 26, 2018
Merged

Add more validations #329

merged 2 commits into from
Feb 26, 2018

Commits on Feb 26, 2018

  1. Add the ability to validate multiple rules files 

    Allow multiple -V arguments just as we do with multiple -r arguments.
    @mstemm
    mstemm committed Feb 26, 2018
    Configuration menu
    Copy the full SHA
    e785198 View commit details
    Browse the repository at this point in the history

  2. With verbose output, print dangling macros/lists 

    Start tracking whether or not a given macro/list is actually used when
compiling the set of rules. Every macro/list has an attribute used,
which defaults to false and is set to true whenever it is referred to in
a macro/rule/list.

When run with -v, any macro/list that still has used=false results in a
warning message.

Also, it turns out the fix for
#197 wasn't being applied to
macros. Fix that.
    @mstemm
    mstemm committed Feb 26, 2018
    Configuration menu
    Copy the full SHA
    58c5119 View commit details
    Browse the repository at this point in the history