Page MenuHomePhabricator

Support including files by shebang in .arclint
Open, LowPublic

Description

Right now .arclint only supports include/exclude by path. We'd like to omit extensions on executables (e.g. https://google.github.io/styleguide/shell.xml?showone=File_Extensions#File_Extensions), but that then means arc lint won't know to lint #!/usr/bin/env python as a python script or #!/bin/bash as a bash script.

It'd be nice to be able to lint based on shebang, maybe something like {"shebang": "#!/usr/bin/env python"}.

Event Timeline

epriestley added a subscriber: epriestley.

This is probably better implemented as "content" ("content matches regexp...") than the narrower "shebang".

eadler added a project: Restricted Project.Jun 30 2016, 6:40 PM