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
DM-35347: Switch to pyproject.toml #46
Conversation
c3a02da
to
d5a4b04
Compare
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.
Looks good. Thanks for sorting out black/isort. Can you also please add a pre-commit configuration?
requires = [ | ||
"setuptools >= 46.6.0", | ||
"setuptools", | ||
"lsst-versions >= 1.3.0", | ||
"wheel", | ||
"pybind11 >= 2.5.0", | ||
"numpy >= 1.18", |
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.
Is this because we need the numpy C header files for the build with pybind11?
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.
Yes.
line-length = 110 | ||
target-version = ["py38"] | ||
|
||
[tool.isort] |
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.
Oh, we hadn't run isort or black before. That explains the failures. Please fix the formatting problems.
3c945a0
to
6aa91cf
Compare
f01f86e
to
4b97dad
Compare
Reordered commits to have all black/isort changes in one. Added |
No description provided.