Skip to content
This repository was archived by the owner on Dec 4, 2023. It is now read-only.

Fixed tabs #36 #37

Closed
wants to merge 1 commit into from
Closed

Fixed tabs #36 #37

wants to merge 1 commit into from

Conversation

stuertz
Copy link

@stuertz stuertz commented Dec 6, 2017

make the code pep8 compliant
fixes #36

make the code pep8 compliant
@jaraco
Copy link
Contributor

jaraco commented Feb 10, 2018

PEP 8 does allow for tabs... and converting the project resets the blame history. Plus, I'd prefer to keep it tabs. Thanks anyhow for suggesting the change.

@jaraco jaraco closed this Feb 10, 2018
jaraco added a commit that referenced this pull request Feb 12, 2021
* Replace pep517.build with build

Resolves #30

* Prefer simple usage

Co-authored-by: Jason R. Coombs <jaraco@jaraco.com>
jaraco added a commit that referenced this pull request Feb 12, 2021
* Use `extend-ignore` in flake8 config

This option allows to add extra ignored rules to the default list
instead of replacing it.

The default exclusions are: E121, E123, E126, E226, E24, E704,
W503 and W504.

Fixes #28.

Refs:
* https://github.com/pypa/setuptools/pull/2486/files#r541943356
* https://flake8.pycqa.org/en/latest/user/options.html#cmdoption-flake8-extend-ignore
*
https://flake8.pycqa.org/en/latest/user/options.html#cmdoption-flake8-ignore

* Enable complexity limit. Fixes jaraco/skeleton#34.

* Replace pep517.build with build (#37)

* Replace pep517.build with build

Resolves #30

* Prefer simple usage

Co-authored-by: Jason R. Coombs <jaraco@jaraco.com>

* Use license_files instead of license_file in meta (#35)

Singular `license_file` is deprecated since wheel v0.32.0.

Refs:
* https://wheel.readthedocs.io/en/stable/news.html
* https://wheel.readthedocs.io/en/stable/user_guide.html#including-license-files-in-the-generated-wheel-file

Co-authored-by: Jason R. Coombs <jaraco@jaraco.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Code not pep8 compliant due to the usage of tabs instead of spaces
2 participants