Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue-reporter-by-user #7

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
35 changes: 35 additions & 0 deletions issue-reported-by-customer.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
_Issues aren't enabled for **codeclimate/pep8**._

_Opening a PR describing the issue._

----

When using a **tox.ini** on an [oss repo](https://github.com/trailofbits/manticore) with:

```
[pep8]
statistics = True
ignore = E265
exclude = docs/,examples/,scripts/,tests/
count = False
```

Code Climate CLI shows:
```
# CODECLIMATE_DEBUG=1 docker run --interactive --tty --rm --env CODECLIMATE_CODE="$PWD" --volume "$PWD":/code --volume /var/run/docker.sock:/var/run/docker.sock --volume /tmp/cc:/tmp/cc codeclimate/codeclimate analyze
Starting analysis
Running structure: Done!
Running duplication: Done!
Running pep8: Done!
error: (CC::CLI::Analyze::EngineFailure) engine pep8 failed with status 99 and stderr
engine produced invalid output: {:message=>"Invalid JSON", :output=>"\n2 E266 too many leading '#' for block comment\n1 E302 expected 2 blank lines, found 1\n11 E402 module level import not at top of file\n1742 E501 line too long (103 > 79 characters)\n1 E704 multiple statements on one line (def)\n33 E712 comparison to False should be 'if cond is False:' or 'if not cond:'\n1 E731 do not assign a lambda expression, use a def\n4 W602 deprecated form of raising exception\n"}
```


If he set `statistics` to `False` in tox.ini, then the codeclimate run completes successfully.

- He says, "It looks like the parser did not account for the summary statistics option."

---

repo: https://github.com/trailofbits/manticore