Skip to content

Commit

Permalink
[#238] fix old config and add msg in deprecated rule
Browse files Browse the repository at this point in the history
  • Loading branch information
Euen committed Jul 3, 2015
1 parent a650860 commit dd361e6
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 4 deletions.
8 changes: 4 additions & 4 deletions config/old/elvis-test-rule-config-list.config
Original file line number Diff line number Diff line change
Expand Up @@ -29,13 +29,13 @@
},
#{dirs => ["."],
filter => "Makefile",
rules => [{elvis_project, no_deps_master_erlang_mk, #{ignore => []}},
{elvis_project, protocol_for_deps_erlang_mk, #{ignore => []}}]
rules => [{elvis_project, no_deps_master_erlang_mk, []},
{elvis_project, protocol_for_deps_erlang_mk, []}]
},
#{dirs => ["."],
filter => "rebar.config",
rules => [{elvis_project, no_deps_master_rebar, #{ignore => []}},
{elvis_project, protocol_for_deps_rebar, #{ignore => []}}]
rules => [{elvis_project, no_deps_master_rebar, []},
{elvis_project, protocol_for_deps_rebar, []}]
},
#{dirs => ["."],
filter => "elvis.config",
Expand Down
4 changes: 4 additions & 0 deletions src/elvis_project.erl
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,8 @@
protocol_for_deps_erlang_mk_config()) ->
[elvis_result:item()].
git_for_deps_erlang_mk(Config, Target, RuleConfig) ->
elvis_utils:error_prn("This rule has been deprecated please use "
"'protocol_for_deps_erlang_mk'."),
protocol_for_deps_erlang_mk(Config, Target, RuleConfig).

-spec protocol_for_deps_erlang_mk(elvis_config:config(),
Expand All @@ -60,6 +62,8 @@ protocol_for_deps_erlang_mk(_Config, Target, RuleConfig) ->
protocol_for_deps_rebar_config()) ->
[elvis_result:item()].
git_for_deps_rebar(Config, Target, RuleConfig) ->
elvis_utils:error_prn("This rule has been deprecated please use "
"'protocol_for_deps_rebar'."),
protocol_for_deps_rebar(Config, Target, RuleConfig).

-spec protocol_for_deps_rebar(elvis_config:config(),
Expand Down

0 comments on commit dd361e6

Please sign in to comment.