diff options
| author | Mike McQuaid | 2017-07-02 14:45:29 +0100 |
|---|---|---|
| committer | GitHub | 2017-07-02 14:45:29 +0100 |
| commit | 2b3717a5a0452271bb7dcfcdfa2680714ac99c2a (patch) | |
| tree | ffb54dbcf971a4fd9fea527c439ca286010e9445 /Library/Homebrew/dev-cmd | |
| parent | eb110e94f002966ffb241e49ca6154fdae601362 (diff) | |
| parent | e6938d920165e3be872366216d618ca01da83fbb (diff) | |
| download | brew-2b3717a5a0452271bb7dcfcdfa2680714ac99c2a.tar.bz2 | |
Merge pull request #2842 from GauthamGoli/audit_only_cops_option
audit: Don't run audit methods when `--only-cops` option is passed
Diffstat (limited to 'Library/Homebrew/dev-cmd')
| -rw-r--r-- | Library/Homebrew/dev-cmd/audit.rb | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/Library/Homebrew/dev-cmd/audit.rb b/Library/Homebrew/dev-cmd/audit.rb index 5b2edfcf6..443a45816 100644 --- a/Library/Homebrew/dev-cmd/audit.rb +++ b/Library/Homebrew/dev-cmd/audit.rb @@ -75,16 +75,18 @@ module Homebrew only_cops = ARGV.value("only-cops").to_s.split(",") except_cops = ARGV.value("except-cops").to_s.split(",") + if !only_cops.empty? && !except_cops.empty? odie "--only-cops and --except-cops cannot be used simultaneously!" - elsif (!only_cops.empty? || !except_cops.empty?) && strict - odie "--only-cops/--except-cops and --strict cannot be used simultaneously" + elsif (!only_cops.empty? || !except_cops.empty?) && (strict || ARGV.value("only")) + odie "--only-cops/--except-cops and --strict/--only cannot be used simultaneously" end options = { fix: ARGV.flag?("--fix"), realpath: true } if !only_cops.empty? options[:only_cops] = only_cops + ARGV.push("--only=style") elsif !except_cops.empty? options[:except_cops] = except_cops elsif !strict |
