Skip to content

Go back to official pyo3-log - #2172

Merged
oleflb merged 3 commits into
HULKs:mainfrom
oleflb:remove-pyo3-fork
Jan 15, 2026
Merged

Go back to official pyo3-log#2172
oleflb merged 3 commits into
HULKs:mainfrom
oleflb:remove-pyo3-fork

Conversation

@oleflb

@oleflb oleflb commented Jan 8, 2026

Copy link
Copy Markdown
Contributor

Why? What?

pyo3-log updated to pyo3 0.27, so we don't have to use my fork anymore.

Fixes #

ToDo / Known Issues

If this is a WIP describe which problems are to be fixed.

Ideas for Next Iterations (Not This PR)

If there are some improvements that could be done in a next iteration, describe them here.

How to Test

Describe how to test your changes. (For the reviewer)

@oleflb oleflb added the is:Cleanup Nothing new here, just tidying up. label Jan 8, 2026
@github-project-automation github-project-automation Bot moved this to Request for Review in Development Jan 8, 2026
@oleflb oleflb added the is:Easy label Jan 8, 2026
@oleflb
oleflb enabled auto-merge January 8, 2026 18:32

@knoellle knoellle left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wanted to test this, but my mujoco-rust-server didn't rebuild when switching between your commit and the one its based on.
Please add the top level Cargo.toml/lock to the cache relevant paths in tools/mujoco-simulator/mujoco-rust-server/pyproject.toml.

@knoellle

knoellle commented Jan 9, 2026

Copy link
Copy Markdown
Member

Worked fine when manually rebuilding though

@oleflb

oleflb commented Jan 13, 2026

Copy link
Copy Markdown
Contributor Author

Wanted to test this, but my mujoco-rust-server didn't rebuild when switching between your commit and the one its based on. Please add the top level Cargo.toml/lock to the cache relevant paths in tools/mujoco-simulator/mujoco-rust-server/pyproject.toml.

Done

@knoellle

Copy link
Copy Markdown
Member

Taplo still isn't happy with your 2525ad1 😅

@knoellle knoellle left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Schaut mich gut an

@oleflb
oleflb added this pull request to the merge queue Jan 15, 2026
Merged via the queue into HULKs:main with commit e43449d Jan 15, 2026
22 checks passed
@oleflb
oleflb deleted the remove-pyo3-fork branch January 15, 2026 20:37
@github-project-automation github-project-automation Bot moved this from Request for Review to Done in Development Jan 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

is:Cleanup Nothing new here, just tidying up. is:Easy

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants