* Remove shared autolint config files
* readthedocs-common submodule added
* Add proper symlinks to all the shared linting config
* Update submodule for unhidden files
* Recover prospector-more.yml since it's specific for this repo
* Config files for auto linting
autoflake, autopep8, docformatter, isort, yapf
* readthedocs.projects.views.base with auto styling applied
* readthedocs.core.models auto styling applied
* readthedocs.builds.urls with auto styling applied
* Do not make multi line summaries
* Improve isort settings
* Use 80 columns as maximum
* pre-commit config file
Use the `pre-commit` tool to run some linting commands before each
commit:
http://pre-commit.com/
* Update yapf config file to 0.20.0
* Add i18n functions to yapf config
* Remove django as a special section for isort
* Rename config to be automatically discovered
* Add annoying as a third party lib for isort
* Use the new config for all the files modified as a test
I had to add a couple of "trailing commas" to keep the format as we want
* Install additional dependencies for each of the hooks
Remove the auto-lint.txt requirements file since it's not needed anymore
* Use default config name for autopep8
* Include autoflake as pre-commit hook
* Exclude D105 from flake8
D105: Missing docstring in magic method
* Minor fixes from flake8
* autoflake with --in-place
* Arguments for docformatter
* Autofix D203 with yapf
D203 / 1 blank line required before class docstring
* Ignore local_settings.py on prospector
This lints project code to medium strictness and conforms to pep257. This uses
some additional options to reduce false errors, which I'll add in a separate PR.
This should pass our normal linting without a problem, except the minor change
to unused variable name regex.
This adds `__` to the default list of `_` and `dummy*` for unused arguments. I
prefer the default of `_`, but that overlaps with the translation helper `_()`.
I'd be happy to rethink a scheme here.
Otherwise pylint will raise a warning if this is used:
logger.debug('Logging value: {value}'.format(value=123))
And recommends using:
logger.debug('Logging value: %(value)s', value=123)
Anthony
Manuel Kaufmann
Alex Chan
Fatema Boxwala
Eric Holscher
Gregor Müllegger