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

AF-137 add help option #28

Merged
merged 3 commits into from
May 25, 2018
Merged

AF-137 add help option #28

merged 3 commits into from
May 25, 2018

Conversation

toddbeckman-wf
Copy link
Contributor

@toddbeckman-wf toddbeckman-wf commented May 25, 2018

Problem

This executable has no usage documentation.

Solution

Add it.

? pub run dependency_validator -h 
-h, --[no-]help                    Displays this info.
    --[no-]verbose                 Display extra information for debugging.
-i, --ignore                       Comma-delimited list of packages to ignore from validation.
-x, --exclude-dir                  Comma-delimited list of directories to exclude from validation.
    --[no-]fatal-pins              Whether to fail on dependency pins.
                                   (defaults to on)

    --[no-]fatal-under-promoted    Whether to fail on dependencies that are in `dev_dependencies` that should be in `dependencies`.
                                   (defaults to on)

    --[no-]fatal-over-promoted     Whether to fail on dependencies that are in `dependencies` that should be in `dev_dependencies`.
                                   (defaults to on)

    --[no-]fatal-missing           Whether to fail on dependencies that are missing from `dependencies`.
                                   (defaults to on)

    --[no-]fatal-dev-missing       Whether to fail on dependencies that are missing from `dev_dependencies`.
                                   (defaults to on)

    --[no-]fatal-unused            Whether to fail on dependencies in the `pubspec.yaml` that are never used in any of:
                                   - bin/
                                   - lib/
                                   - examples/
                                   - test/
                                   - tool/
                                   - web/
                                   (defaults to on)

Testing

  • pub run dependency_validator -h
  • pub run dependency_validator --help

@aviary2-wf
Copy link

Security Insights

No security relevant content was detected by automated scans.

Action Items

  • Review PR for security impact; comment "security review required" if needed or unsure
  • Verify aviary.yaml coverage of security relevant code

Questions or Comments? Reach out on HipChat: InfoSec Forum.

@toddbeckman-wf toddbeckman-wf mentioned this pull request May 25, 2018
2 tasks
@evanweible-wf
Copy link
Contributor

+10

  • pub run dependency_validator -h
  • pub run dependency_validator --help
  • CI passes

QA +1

  • Dev +1s
  • +10 tested
  • Rosie is happy

@Workiva/release-management-p

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.

6 participants