Skip to content

Conversation

@asukaminato0721
Copy link
Contributor

Which issue does this PR close?

Closes #6479.

Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

user can use Path in root.

@asukaminato0721 asukaminato0721 changed the title try fix feat(bindings/python): allow Path in root Aug 7, 2025
@asukaminato0721
Copy link
Contributor Author

ohh, the new clippy warning, I don't want to fix them in this pr.

@asukaminato0721
Copy link
Contributor Author

tests/test_operator_construction.py::test_operator_new_with_path PASS

@asukaminato0721 asukaminato0721 marked this pull request as ready for review August 7, 2025 18:15
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. releases-note/feat The PR implements a new feature or has a title that begins with "feat" labels Aug 7, 2025
@asukaminato0721 asukaminato0721 self-assigned this Aug 7, 2025
@kingsword09
Copy link
Contributor

ohh, the new clippy warning, I don't want to fix them in this pr.

This has already been resolved and merged into main.

@asukaminato0721
Copy link
Contributor Author

PyErr { type: <class 'TypeError'>, value: TypeError("'PosixPath' object cannot be converted to 'PyString'"), traceback: None }

Need some fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/feat The PR implements a new feature or has a title that begins with "feat" size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

new feature: Python should accept Path for root argument

4 participants