Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Replace nox by pixi #5898

Closed
abey79 opened this issue Apr 10, 2024 · 2 comments · Fixed by #7508
Closed

Replace nox by pixi #5898

abey79 opened this issue Apr 10, 2024 · 2 comments · Fixed by #7508
Labels
🐍 Python API Python logging API 🚢 CI
Milestone
Spring Cleaning

Comments

@abey79
Copy link
Member

abey79 commented Apr 10, 2024 •
edited
Loading

The purpose of nox is to run test against various version of python. This could be implemented in pixi.
@abey79
Copy link
Member Author

abey79 commented Apr 10, 2024 •
edited by emilk
Loading

@abey79 abey79 mentioned this issue Apr 10, 2024
Open
@emilk emilk added 🐍 Python API Python logging API 🚢 CI labels Apr 12, 2024
@emilk
Copy link
Member

emilk commented Apr 16, 2024 •
edited
Loading

Why should we do this?

@emilk emilk added this to the Spring Cleaning milestone Apr 24, 2024
@Wumpf Wumpf mentioned this issue Sep 25, 2024
Merged
8 tasks
Wumpf added a commit that referenced this issue Sep 25, 2024
@Wumpf
Remove unused nox dev dependency & mentions (#7508)
199e4b1 
### What

* Fixes #5898

We stopped using `nox` for python testing quite a while ago.

### Checklist
* [x] pass main ci
* [x] confirm that `pixi run py-test` works fine locally 	
* [x] I have read and agree to [Contributor
Guide](https://github.com/rerun-io/rerun/blob/main/CONTRIBUTING.md) and
the [Code of
Conduct](https://github.com/rerun-io/rerun/blob/main/CODE_OF_CONDUCT.md)
* [x] I've included a screenshot or gif (if applicable)
* [x] I have tested the web demo (if applicable):
* Using examples from latest `main` build:
[rerun.io/viewer](https://rerun.io/viewer/pr/7508?manifest_url=https://app.rerun.io/version/main/examples_manifest.json)
* Using full set of examples from `nightly` build:
[rerun.io/viewer](https://rerun.io/viewer/pr/7508?manifest_url=https://app.rerun.io/version/nightly/examples_manifest.json)
* [x] The PR title and labels are set such as to maximize their
usefulness for the next release's CHANGELOG
* [x] If applicable, add a new check to the [release
checklist](https://github.com/rerun-io/rerun/blob/main/tests/python/release_checklist)!
* [x] If have noted any breaking changes to the log API in
`CHANGELOG.md` and the migration guide

- [PR Build Summary](https://build.rerun.io/pr/7508)
- [Recent benchmark results](https://build.rerun.io/graphs/crates.html)
- [Wasm size tracking](https://build.rerun.io/graphs/sizes.html)

To run all checks from `main`, comment on the PR with `@rerun-bot
full-check`.
@abey79 abey79 mentioned this issue Sep 25, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🐍 Python API Python logging API 🚢 CI
Projects
None yet
Milestone
Spring Cleaning
Development

Successfully merging a pull request may close this issue.

Remove unused nox dev dependency & mentions rerun-io/rerun
2 participants
@emilk @abey79