Skip to content

Commit

Permalink
unbreak Puppet linter: use line format string instead of linenumber
Browse files Browse the repository at this point in the history
This diff updates ArcanistPuppetLintLinter.php to be compatible with the latest `puppet-lint`.

Based on the change made by the `puppet-lint` maintainers in Issue rodjek/puppet-lint#539 and commithttps://github.com/rodjek/puppet-lint/pull/540 the `linenumber` format string is no longer supported.

While the issue was already reported in https://secure.phabricator.com/T10038 and https://secure.phabricator.com/T12169 it has never been fixed.
  • Loading branch information
Radek Zajic committed Apr 1, 2020
1 parent b1a712a commit 987185c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/lint/linter/ArcanistPuppetLintLinter.php
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ protected function getMandatoryFlags() {
return array(
'--error-level=all',
sprintf('--log-format=%s', implode('|', array(
'%{linenumber}',
'%{line}',
'%{column}',
'%{kind}',
'%{check}',
Expand Down

0 comments on commit 987185c

Please sign in to comment.