Luke O'Malley

Results 33 issues of Luke O'Malley

From @ievans > EXE002 is 1 indexed but EXE001 is 0 indexed...should both be one indexed. It's annoying for [my] scripts that depend on the line number in Bento's output....

bug
tool:flake8
feature:program-analysis

## Overview Flake8 C901 is missing a rename and may be a best practice / stylistic. ```shell flake8 C901 > migrations/201802021027_af3f5579c84d_add_cascades.py:24 ╷ 24│ def upgrade(): ╵ = 'upgrade' is too...

bug
tool:flake8
feature:program-analysis

**Is your feature request related to a problem? Please describe.** Bento currently supports disabling all checks per-file (`.bentoignore`) or project-wide (`bento disable check`). While this is sufficient in most cases,...

request:feature
feature:check
fixed

**Is your feature request related to a problem? Please describe.** tflint looks promising and Terraform checking has been a common Bento request. https://github.com/terraform-linters/tflint **Describe the solution you’d like** Run tflint...

request:enhancement
feature:check

## Overview The Bandit check `import_xml-etree` contains a `_` instead of a `-`. ## Steps to Reproduce 1. `$ bento init` 2. `cat .bento/config.yml` ## Possible Solution The is present...

bug
tool:bandit
feature:check
starter-project

## Overview Ran `bento check` with a corrupted cache. Bento fails to parse the cache and hard fails, telling me to run `bento init --clean` which addresses a different problem....

bug
feature:check

> It would be great if Bento had some checks to measure code quality, for example: > >  - Cognitive Complexity >  - Cyclomatic Complexity >  - Anything that could...

request:feature
feature:check
discussion-required

## Overview Flake8 triggers on comments in the https://github.com/pantsbuild/pants project: ```shell flake8 undefined-name > build-support/bin/ci.py:215 ╷ 215│ if self == self.v2_remote and oauth_token_path is None: # type: ignore[comparison-overlap] # issues...

bug
tool:flake8
feature:check

r2c.flask is only showing single lines of syntactic context for multi-line findings. ```bash r2c.flask unescaped-file-extension > tests/test_templating.py:96 ╷ 96│ return flask.render_template( ╵ = Flask does not automatically escape Jinja templates...

bug
feature:check
tool:r2c.flask

This was initially detected by @ievans in two places in his workflow: locally and in CI: > @ievans: DUO102 should be disabled. Also r2c.flake8.E117. > @dlukeomalley: They should be in...

request:enhancement
discussion-required