Switch to pixi and support windows #1
Conversation
| - name: Publish package on TestPyPi | ||
| uses: pypa/gh-action-pypi-publish@81e9d935c883d0b210363ab89cf05f3894778450 | ||
| with: | ||
| repository-url: https://test.pypi.org/legacy/ | ||
| - name: Publish package on PyPi | ||
| uses: pypa/gh-action-pypi-publish@81e9d935c883d0b210363ab89cf05f3894778450 |
There was a problem hiding this comment.
You need to create a first package manually and upload it to testpypi and pypi.
Also, you need to add trusted publishers s.t. auth via OIDC works
Co-authored-by: Pavel Zwerschke <[email protected]>
Co-authored-by: Pavel Zwerschke <[email protected]>
Co-authored-by: Pavel Zwerschke <[email protected]>
adityagoel4512
force-pushed
the
pixi
branch
2 times, most recently
from
96b1c49
to
ad0f757
Compare
README.md
Outdated
|
|
||
| # Set up development environment | ||
| pixi shell | ||
| pre-commit install |
There was a problem hiding this comment.
this won't always work since pre-commit is not part of the default env. why not use pixi run pre-commit-install which automatically uses the lint env?
There was a problem hiding this comment.
Fair enough - wanted to keep the set up familiar to non pixi users. I think I'll create a dev env with everything bundled into it.
README.md
Outdated
| pre-commit install | ||
| pip install --no-build-isolation -e . | ||
| pip install -e . |
There was a problem hiding this comment.
this will use build isolation, i.e. use setuptools-scm outside of the pixi environment. you would need --no-build-isolation for this. Also, you should specify --no-deps to not use pypi dependencies from pyproject.toml and instead use the conda dependencies from your pixi environment
pyproject.toml
Outdated
| [tool.pixi.project] | ||
| channels = ["conda-forge"] | ||
| platforms = ["osx-arm64"] | ||
|
|
||
| [tool.pixi.pypi-dependencies] | ||
| ndonnx = { path = ".", editable = true } | ||
|
|
||
| [tool.pixi.tasks] |
There was a problem hiding this comment.
you should specify these things in pixi.toml (or use pyproject.toml solely). also, i wouldn't recommend ndonnx = { path = ".", editable = true } as of now: prefix-dev/pixi#1046
adityagoel4512
force-pushed
the
pixi
branch
2 times, most recently
from
f6931d2
to
157c865
Compare
adityagoel4512
force-pushed
the
pixi
branch
2 times, most recently
from
a80f69b
to
6d2d82a
Compare
adityagoel4512
force-pushed
the
pixi
branch
10 times, most recently
from
4c3e61e
to
734b2eb
Compare
Checklist
CHANGELOG.rstentry