-
Notifications
You must be signed in to change notification settings - Fork 47
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
Update package metadata #75
Changes from 4 commits
c8a84dc
3c2d466
c217221
e670d07
46926ca
ed3c1e8
3276c7e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,19 +1,68 @@ | ||
[build-system] | ||
requires = ["setuptools >= 44", "wheel >= 0.30", "setuptools_scm[toml]>=5"] | ||
build-backend = "setuptools.build_meta" | ||
build-backend = "hatchling.build" | ||
requires = ["hatchling>=0.22.0", "hatch-vcs"] | ||
|
||
[project] | ||
name = "platformdirs" | ||
description = 'A small Python module for determining appropriate platform-specific dirs, e.g. a "user data dir".' | ||
readme = "README.rst" | ||
license = "MIT" | ||
maintainers = [ | ||
{ name = "Bernát Gábor", email = "gaborjbernat@gmail.com" }, | ||
{ name = "Julian Berman", email = "Julian@GrayVines.com" }, | ||
{ name = "Ofek Lev", email = "oss@ofek.dev" }, | ||
{ name = "Ronny Pfannschmidt", email = "opensource@ronnypfannschmidt.de" }, | ||
] | ||
urls.Documentation = "https://platformdirs.readthedocs.io" | ||
urls.Homepage = "https://github.com/platformdirs/platformdirs" | ||
urls.Source = "https://github.com/platformdirs/platformdirs" | ||
urls.Tracker = "https://github.com/platformdirs/platformdirs/issues" | ||
requires-python = ">=3.7" | ||
optional-dependencies.test = [ | ||
"appdirs==1.4.4", | ||
"pytest>=6", | ||
"pytest-cov>=2.7", | ||
"pytest-mock>=3.6", | ||
] | ||
optional-dependencies.docs = [ | ||
"furo>=2021.7.5b38", | ||
"proselint>=0.10.2", | ||
"sphinx>=4", | ||
"sphinx-autodoc-typehints>=1.12", | ||
] | ||
keywords = ["application", "cache", "directory", "log", "user"] | ||
classifiers = [ | ||
"Development Status :: 5 - Production/Stable", | ||
"Intended Audience :: Developers", | ||
"License :: OSI Approved :: MIT License", | ||
"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 :: Libraries :: Python Modules", | ||
] | ||
dynamic = ["version"] | ||
|
||
[tool.hatch] | ||
build.hooks.vcs.template = ''' | ||
"""Version information""" | ||
|
||
__version__ = "{version}" | ||
__version_info__ = {version_tuple} | ||
''' | ||
build.hooks.vcs.version-file = "src/platformdirs/version.py" | ||
build.targets.sdist.include = ["/src", "/tests", "/tox.ini"] | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Hmm, why do we need these? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Well there was no There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Because setuptools-scm packages every VCS file by default into the sdist, there's no need for a There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Everything by default too, removing now There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can you define everything? Everything that's version-controlled and everything in the working directory are very different 😆 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ha true, the former: https://ofek.dev/hatch/latest/plugins/builder/#default-file-selection_1 |
||
version.source = "vcs" | ||
|
||
[tool.black] | ||
line-length = 120 | ||
|
||
[tool.isort] | ||
profile = "black" | ||
known_first_party = ["platformdirs"] | ||
|
||
[tool.setuptools_scm] | ||
write_to = "src/platformdirs/version.py" | ||
write_to_template = """ | ||
\"\"\" Version information \"\"\" | ||
|
||
__version__ = "{version}" | ||
__version_info__ = {version_tuple} | ||
""" |
This file was deleted.
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 you add https://github.com/tox-dev/pyproject-fmt instead now?
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.
CI failing
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.
Will have a look 👍