-
-
Notifications
You must be signed in to change notification settings - Fork 258
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
Document binary and library in a single package by entrypoint workaround #1565
Conversation
✅ Deploy Preview for maturin-guide ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
This is awesome! |
File path on Windows is ridiculously complicated, maybe we should simplify the test case to not assert file paths. |
The tests are passing now, i've removed checks for platforms with inconsistent behaviors. I think the current |
7a84ba1
to
cfcc8e2
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.
bors r+
1565: Document binary and library in a single package by entrypoint workaround r=messense a=konstin Inspired by #368 (comment) Preview: https://deploy-preview-1565--maturin-guide.netlify.app/bindings.html#both-binary-and-library CC `@nanthony007` Co-authored-by: konstin <[email protected]>
Build failed: |
bors retry looks like a network error: https://github.com/PyO3/maturin/actions/runs/4760507648/jobs/8460910927 |
1565: Document binary and library in a single package by entrypoint workaround r=messense a=konstin Inspired by #368 (comment) Preview: https://deploy-preview-1565--maturin-guide.netlify.app/bindings.html#both-binary-and-library CC `@nanthony007` Co-authored-by: konstin <[email protected]>
Build failed: |
bors r+ discovering more strange platform behavior as i go |
1565: Document binary and library in a single package by entrypoint workaround r=konstin a=konstin Inspired by #368 (comment) Preview: https://deploy-preview-1565--maturin-guide.netlify.app/bindings.html#both-binary-and-library CC `@nanthony007` Co-authored-by: konstin <[email protected]>
Build failed: |
bors retry |
Build succeeded: |
Inspired by #368 (comment)
Preview: https://deploy-preview-1565--maturin-guide.netlify.app/bindings.html#both-binary-and-library
CC @nanthony007