Page MenuHomePhabricator

Update puppet-lint format string to not use deprecated `linenumber`.
Needs ReviewPublic

Authored by oc243 on Mar 24 2017, 12:07 PM.
Tags
None
Referenced Files
F13081261: D17548.diff
Wed, Apr 24, 4:49 PM
Unknown Object (File)
Mon, Apr 22, 5:44 AM
Unknown Object (File)
Sat, Apr 20, 4:48 PM
Unknown Object (File)
Wed, Apr 17, 4:56 AM
Unknown Object (File)
Mar 23 2024, 9:14 AM
Unknown Object (File)
Feb 12 2024, 6:05 PM
Unknown Object (File)
Jan 25 2024, 5:30 AM
Unknown Object (File)
Jan 14 2024, 6:16 PM
Subscribers
This revision can not be accepted until the required legal agreements have been signed.

Details

Summary

puppet-lint's format string uses line rather than linenumber now to print
line numbers. Using linenumber produces a warning:
DEPRECATION: Please use %{line} instead of %{linenumber}.

Test Plan

Tested on MacOS with puppet-lint v2.

Diff Detail

Repository
rARC Arcanist
Branch
master
Lint
Lint Passed
Unit
Test Failures
Build Status
Buildable 16121
Build 21392: arc lint + arc unit

Unit TestsFailed

TimeTest
1,629 msArcanistFlake8LinterTestCase::Unknown Unit Message ("")
In 'undefined.lint-test', expected lint to raise error on line 3 at char 1, but no error was raised. Actually raised: No messages.
39 msArcanistChmodLinterTestCase::Unknown Unit Message ("")
5 assertions passed.
0 msArcanistClosureLinterTestCase::Unknown Unit Message ("")
1 assertion passed.
0 msArcanistCpplintLinterTestCase::Unknown Unit Message ("")
1 assertion passed.
27 msArcanistFilenameLinterTestCase::Unknown Unit Message ("")
5 assertions passed.
View Full Test Results (1 Failed · 30 Passed · 23 Skipped)

Content Hidden

The content of this revision is hidden until the author has signed all of the required legal agreements.