Skip to content

Add tests for the generated documentation#53

Draft
antalsz wants to merge 6 commits intomainfrom
52-add-documentation-tests
Draft

Add tests for the generated documentation#53
antalsz wants to merge 6 commits intomainfrom
52-add-documentation-tests

Conversation

@antalsz
Copy link
Copy Markdown
Contributor

@antalsz antalsz commented Jul 9, 2024

Closes #52. Leaving as draft because it's not clear the best way to do this. Extends #51.

antalsz added 5 commits July 8, 2024 17:34
We have to specify a specific OS target because until PyO3 v0.22, PyO3
transitively depends on the [rust-ctor](https://crates.io/crates/ctor) crate,
which generates different output on different OSes.  Once we're doing *that*, we
have to specify a specific Python ABI so that PyO3 doesn't get alarmed about
cross-compilation.  This is all a minor headache.

One concern about this is that everyone testing locally has to run `rustup
target add x864_64-unknown-linuxgnu`.  CI doesn't, because I deliberately picked
the triple that was running on CI, but any local development that isn't on that
OS does.  Is this worth it?
@antalsz antalsz force-pushed the 52-add-documentation-tests branch from 4c1ad71 to dcda90e Compare July 9, 2024 00:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add test cases for the generated documentation

1 participant