django-rest-framework-datatables
django-rest-framework-datatables copied to clipboard
Replace setup and requirements files with pyproject.toml
- Fixes #135
- Moves version number into toml file
- Bumps version to 0.7.1
- Not including poetry.lock per Poetry recommendations that it is not needed for libraries (https://python-poetry.org/docs/basic-usage/#commit-your-poetrylock-file-to-version-control)
- Also fixes minor sphinx block rendering issue
Hi @jacklinke , thanks for your contribution !
I'm really not sure I want to force the usage of poetry.
Correct me if I'm wrong (I've never used poetry), but this would break current installations that use pip.
Could it be possible to keep pip support along with poetry ?
I agree with @izimobil - could we keep pip support?
Could it be possible to keep pip support along with poetry ?
If you mean to keep the ability to do pip install against the repo, that still works due to the build backend:
https://github.com/jacklinke/django-rest-framework-datatables/blob/9b7bb47d280db56536f6d1caff6ac224da42b4e9/pyproject.toml#L59-L61
You can test it with:
pip install https://github.com/jacklinke/django-rest-framework-datatables/archive/add_pyproject_toml.zip
If you really don't like Poetry (which is fine, it's opinionated and not for everyone), there is an alternative to use pyproject.toml using setuptools, which you might find closer to the official standard.
Could it be possible to keep pip support along with poetry ?
If you mean to keep the ability to do
pip installagainst the repo, that still works due to the build backend:https://github.com/jacklinke/django-rest-framework-datatables/blob/9b7bb47d280db56536f6d1caff6ac224da42b4e9/pyproject.toml#L59-L61
You can test it with:
pip install https://github.com/jacklinke/django-rest-framework-datatables/archive/add_pyproject_toml.zipIf you really don't like Poetry (which is fine, it's opinionated and not for everyone), there is an alternative to use
pyproject.tomlusingsetuptools, which you might find closer to the official standard.
To be honest, I don't know / never used Poetry, but reading the proposed pyproject.toml gives me headaches ;)
Would adding something like this (copied from Django's pyproject.toml) be sufficient to address the original issue #135 ?
[build-system]
requires = ['setuptools>=40.8.0']
build-backend = 'setuptools.build_meta'
Maybe I'm getting old, but "If it ain't broke, don't fix it" comes to my mind...