Page MenuHomePhabricator

Add puppet-lint linter
AbandonedPublic

Authored by joshuaspence on Mar 5 2014, 10:34 PM.
Tags
None
Referenced Files
F14875175: D8418.id.diff
Sat, Feb 8, 11:52 AM
Unknown Object (File)
Fri, Jan 24, 10:18 PM
Unknown Object (File)
Thu, Jan 23, 9:51 PM
Unknown Object (File)
Thu, Jan 23, 9:51 PM
Unknown Object (File)
Wed, Jan 22, 12:30 PM
Unknown Object (File)
Mon, Jan 20, 11:36 AM
Unknown Object (File)
Fri, Jan 17, 2:37 PM
Unknown Object (File)
Dec 16 2024, 11:56 PM
Tokens
"Love" token, awarded by dctrwatson.

Details

Reviewers
epriestley
aripringle
Group Reviewers
Blessed Reviewers
Summary

Add ArcanistPuppetLintLinter class to test Puppet manifest files. Relies on the puppet-lint ruby gem.

Test Plan

New test written. Also did manual testing of warnings, errors, and files with no lint messages. Verified that .arcconfig options work as expected.

Diff Detail

Repository
rARC Arcanist
Branch
master
Lint
Lint Passed
Unit
Test Failures

Event Timeline

This generally looks good, but we should make two changes which aren't entirely straightforward:

  • Let's wait for D8387 and use that style (array of flags instead of string).
  • I don't want to add new .arcconfig options, since .arclint will obsolete them. However, .arclint is in a weird limbo right now and we don't have a solid technical approach for deprecating .arcconfig options or letting you use older linters easily without introducing them. Let me come up with a more concrete approach here.

I'll follow up with specific guidance once some of the ambiguity here clears up. Ideally, I'd like to get .arclint pushed forward enough to start recommending it, but I'm not sure how my priorities will line up.

joshuaspence added a reviewer: aripringle.
joshuaspence added a subscriber: joshuaspence.

This is fixed by D8990.