lint: Enable python linters via an array #15219

pull Empact wants to merge 1 commits into bitcoin:master from Empact:python-lint changing 1 files +75 −73

Empact commented at 1:14 AM on January 21, 2019: member

This assures consistent recording of the enabled linters.

This applies the same fix as #15170 to lint-python.sh

lint: Enable python linters via an array

This assures consistent recording of the enabled linters.

948d8f4f10

fanquake added the label Scripts and tools on Jan 21, 2019
practicalswift commented at 2:24 PM on January 21, 2019: contributor

utACK 948d8f4f10c31220ba4b6779cc862e2b6a0af5f6

Much nicer. Thanks for doing this!
in test/lint/lint-python.sh:1 in 948d8f4f10
```
   0 | @@ -1,4 +1,4 @@
   1 | -#!/bin/sh
   2 | +#!/usr/bin/env bash
```
laanwj commented at 3:45 PM on January 21, 2019:

Adding a specific dependency on bash is acceptable, because other linter scripts already rely on bash-isms anyway, I suppose?

Edit: ah yes, I see this was one of the few scripts in lint that didn't have a specific bash shebang.
laanwj commented at 3:45 PM on January 21, 2019: member

utACK 948d8f4f10c31220ba4b6779cc862e2b6a0af5f6
laanwj merged this on Jan 21, 2019
laanwj closed this on Jan 21, 2019
laanwj referenced this in commit 9bad1e0b22 on Jan 21, 2019
Munkybooty referenced this in commit 6795d46e4f on Aug 21, 2021
Munkybooty referenced this in commit 43c6128944 on Aug 23, 2021
Munkybooty referenced this in commit d936c1420c on Aug 24, 2021
Munkybooty referenced this in commit 31e68b8988 on Aug 24, 2021
Munkybooty referenced this in commit 8c7735da6b on Aug 24, 2021
UdjinM6 referenced this in commit 34614f8cff on Aug 24, 2021
Munkybooty referenced this in commit 846473aed2 on Aug 24, 2021
MarcoFalke locked this on Dec 16, 2021

Contributors

Labels