search

plone / plone.recipe.codeanalysis

provides static code analysis for Buildout-based Python projects, including flake8, JSHint, CSS Lint, and other code checks
https://pypi.org/project/plone.recipe.codeanalysis/
11 stars 5 forks source link

Check for git merge leftovers. #179

Open tisto opened 8 years ago

tisto commented 8 years ago

Do we have a plugin that supports checking for merge leftovers such as "======", ">>>>>>", and "<<<<<<"? I know that we used to have a configurable checker in an earlier version. cc @gforcada

hvelarde commented 8 years ago

+1

gforcada commented 8 years ago

I don't recall it right now, but should be trivial to make one.

gforcada commented 8 years ago

I see that there is this flake8 plugin: https://pypi.python.org/pypi/flake8_dodgy/0.0.1 which is done by the guys at landscape.io

Not sure if we want to depend on that one though, as it adds quite a lot of other checks, which may give quite a few false positives (just a gut feeling).

do3cc commented 8 years ago

flake8_dodgy currently checks against passwords, secrets, aws secrets and diffs. Seems to be quite safe. I wanted to suggest to use a pin in setup.py here, but this would prevent anybody from using newer versions.