Page MenuHomeDevCentral

Configure Flake8 linter for Arcanist
ClosedPublic

Authored by dereckson on Feb 14 2023, 19:19.
Tags
None
Referenced Files
F24676886: D2795.id7094.diff
Wed, Mar 4, 16:24
F24671104: D2795.diff
Wed, Mar 4, 10:06
F24668897: D2795.diff
Wed, Mar 4, 07:17
F24666618: D2795.id7093.diff
Wed, Mar 4, 04:37
F24666612: D2795.id7094.diff
Wed, Mar 4, 04:37
F24664524: D2795.id7093.diff
Wed, Mar 4, 01:12
F24664521: D2795.id7094.diff
Wed, Mar 4, 01:12
F24664485: D2795.diff
Wed, Mar 4, 01:10
Subscribers
None

Details

Summary

Allow to specify pillar and other Salt dictionaries as builtins,
so flake8 won't complain anymore those names don't exist.

Lint also the .sls #!py files, as they are written in Python too.

This currently needs to use Nasqueron version of arcanist, production branch:
https://github.com/nasqueron/arcanist/commit/15aa482eacd90fdbd238bc8b7a17c4bbba8a0bdd

Test Plan

arc lint

Diff Detail

Repository
rOPS Nasqueron Operations
Lint
Lint Passed
Unit
No Test Coverage
Branch
main
Build Status
Buildable 4427
Build 4694: arc lint + arc unit

Event Timeline

dereckson created this revision.
This revision is now accepted and ready to land.Feb 14 2023, 19:20
This revision was automatically updated to reflect the committed changes.