Skip to content

Commit

Permalink
Merge pull request #176 from paulo-ferraz-oliveira/fix/warning_explic…
Browse files Browse the repository at this point in the history
…itness

Ease consumption of warning
  • Loading branch information
Brujo Benavides authored Feb 8, 2021
2 parents d702cd6 + 4e8c3b7 commit c4ffeb0
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
7 changes: 5 additions & 2 deletions src/elvis_config.erl
Original file line number Diff line number Diff line change
Expand Up @@ -160,8 +160,11 @@ resolve_files(RuleGroup, Files) ->
FilteredFiles = elvis_file:filter_files(Files, Dirs, Filter, Ignore),
_ = case FilteredFiles of
[] ->
Error = elvis_result:new(warn, "Searching for files in ~p yielded none. "
"Update your configuration.", [Dirs]),
Ruleset = maps:get(ruleset, RuleGroup, undefined),
Error = elvis_result:new(warn, "Searching for files in ~p, for ruleset ~p, "
"with filter ~p, yielded none. "
"Update your configuration.",
[Dirs, Ruleset, Filter]),
ok = elvis_result:print_results([Error]);
_ ->
ok
Expand Down
2 changes: 1 addition & 1 deletion src/elvis_result.erl
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ get_line_num(#{line_num := LineNum}) -> LineNum.

%% Print

-spec print_results(file()) -> ok.
-spec print_results(file() | [elvis_warn()]) -> ok.
print_results(Results) ->
Format = application:get_env(elvis_core, output_format, colors),
print(Format, Results).
Expand Down

0 comments on commit c4ffeb0

Please sign in to comment.