And just for funsies let's open a fourth PR :grin:
It's branched on top of #162, so the diff will be slightly smaller once that's merged.
Ready for review, though I can probably polish it a bit if giving it another look tomorrow.
Had a decently long thonk on whether to just have an --enable and no --disable, or whether to get the full flake8 experience with --extend-[...] flags. Settled on this compromise.
91x does not respect --autofix parameter value properly atm, hence the commented out code with that, would like #159 in order to work on that.
Will probably have some merge conflicts and need rebasing once other PR's get merged
And just for funsies let's open a fourth PR :grin: It's branched on top of #162, so the diff will be slightly smaller once that's merged.
Ready for review, though I can probably polish it a bit if giving it another look tomorrow. Had a decently long thonk on whether to just have an
--enable
and no--disable
, or whether to get the full flake8 experience with--extend-[...]
flags. Settled on this compromise.91x does not respect
--autofix
parameter value properly atm, hence the commented out code with that, would like #159 in order to work on that.Will probably have some merge conflicts and need rebasing once other PR's get merged