-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Improve packaging [PEP 517 + 621] #7076
Merged
Merged
Changes from 7 commits
Commits
Show all changes
13 commits
Select commit
Hold shift + click to select a range
2842e4f
Use isolated build environments
cdce8p 74ae6dd
Use new project metadata format [PEP 621]
cdce8p abd9179
Cleanup unused MANIFEST entries
cdce8p c196772
Update misc files
cdce8p 7798d91
Remove MANIFEST.in
cdce8p 47b8c90
Fix flake8
cdce8p 366d05b
Fix keywords
cdce8p 3edc9f0
Revert "Remove MANIFEST.in"
cdce8p ff074bf
Keep setup.py
cdce8p b089d4a
Add dependabot to custom dictionary
cdce8p e047d47
Move remaining flake8 config to setup.cfg
cdce8p e860338
Remove setup.py
cdce8p d380aca
Update pre-commit config
cdce8p File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,79 @@ | ||
[build-system] | ||
requires = ["setuptools~=62.6", "wheel~=0.37.1"] | ||
build-backend = "setuptools.build_meta" | ||
|
||
[project] | ||
name = "pylint" | ||
license = {text = "GPL-2.0-or-later"} | ||
description = "python code static checker" | ||
readme = "README.rst" | ||
authors = [ | ||
{name = "Python Code Quality Authority", email = "code-quality@python.org"} | ||
] | ||
keywords = ["static code analysis", "linter", "python", "lint"] | ||
classifiers = [ | ||
"Development Status :: 6 - Mature", | ||
"Environment :: Console", | ||
"Intended Audience :: Developers", | ||
"License :: OSI Approved :: GNU General Public License v2 (GPLv2)", | ||
"Operating System :: OS Independent", | ||
"Programming Language :: Python", | ||
"Programming Language :: Python :: 3", | ||
"Programming Language :: Python :: 3 :: Only", | ||
"Programming Language :: Python :: 3.7", | ||
"Programming Language :: Python :: 3.8", | ||
"Programming Language :: Python :: 3.9", | ||
"Programming Language :: Python :: 3.10", | ||
"Programming Language :: Python :: Implementation :: CPython", | ||
"Programming Language :: Python :: Implementation :: PyPy", | ||
"Topic :: Software Development :: Debuggers", | ||
"Topic :: Software Development :: Quality Assurance", | ||
"Topic :: Software Development :: Testing", | ||
] | ||
requires-python = ">=3.7.2" | ||
dependencies = [ | ||
"dill>=0.2", | ||
"platformdirs>=2.2.0", | ||
# Also upgrade requirements_test_min.txt if you are bumping astroid. | ||
# Pinned to dev of next minor update to allow editable installs, | ||
# see https://github.com/PyCQA/astroid/issues/1341 | ||
"astroid>=2.11.6,<=2.12.0-dev0", | ||
"isort>=4.2.5,<6", | ||
"mccabe>=0.6,<0.8", | ||
"tomli>=1.1.0;python_version<'3.11'", | ||
"tomlkit>=0.10.1", | ||
"colorama>=0.4.5;sys_platform=='win32'", | ||
"typing-extensions>=3.10.0;python_version<'3.10'", | ||
] | ||
dynamic = ["version"] | ||
|
||
[project.optional-dependencies] | ||
testutils = ["gitpython>3"] | ||
spelling = ["pyenchant~=3.2"] | ||
|
||
[project.urls] | ||
"Docs: User Guide" = "https://pylint.pycqa.org/en/latest/" | ||
"Source Code" = "https://github.com/PyCQA/pylint" | ||
"What's New" = "https://pylint.pycqa.org/en/latest/whatsnew/2/" | ||
"Bug Tracker" = "https://github.com/PyCQA/pylint/issues" | ||
"Discord Server" = "https://discord.com/invite/Egy6P8AMB5" | ||
"Docs: Contributer Guide" = "https://pylint.pycqa.org/en/latest/development_guide/contributor_guide/index.html" | ||
|
||
[project.scripts] | ||
pylint = "pylint:run_pylint" | ||
pylint-config = "pylint:_run_pylint_config" | ||
epylint = "pylint:run_epylint" | ||
pyreverse = "pylint:run_pyreverse" | ||
symilar = "pylint:run_symilar" | ||
|
||
[tool.setuptools] | ||
license-files = ["LICENSE", "CONTRIBUTORS.txt"] # Keep in sync with setup.cfg | ||
|
||
[tool.setuptools.packages.find] | ||
include = ["pylint*"] | ||
|
||
[tool.setuptools.package-data] | ||
pylint = ["testutils/testing_pylintrc"] | ||
|
||
[tool.setuptools.dynamic] | ||
version = {attr = "pylint.__pkginfo__.__version__"} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't this go in
.flake8
?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's do it the other way round and move the whole flake8 config into
setup.cfg
. In the past, we have also removed.isort
. At some point we can move most / all of these configs intopyproject.toml
.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Strong agree. I don't think
.flake8
support pyproject.toml right now though . But there's other tool we could move already, tox and black probably.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(PyCQA/flake8#234 (comment))
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would recommend to keep
tox
(andpylint
) separate. Both have quite complex configurations. It's much easier to have them in their individual files. We could however moveisort
,pytest
, andmypy
.