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.

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

Add tests for the generated documentation #53

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Commits on Jul 1, 2024

  1. Configuration menu
    Copy the full SHA
    a2fa204 View commit details
    Browse the repository at this point in the history

Commits on Jul 9, 2024

  1. test: move test code into a separate module

    This enables easier manual testing with `cargo expand`, and will enable easier
    snapshot testing later.
    antalsz committed Jul 9, 2024
    Configuration menu
    Copy the full SHA
    f3ca274 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    5710664 View commit details
    Browse the repository at this point in the history
  3. ci: add cargo-expand to CI

    antalsz committed Jul 9, 2024
    Configuration menu
    Copy the full SHA
    cd220fc View commit details
    Browse the repository at this point in the history
  4. ci: satisfy cargo deny

    antalsz committed Jul 9, 2024
    Configuration menu
    Copy the full SHA
    464b746 View commit details
    Browse the repository at this point in the history
  5. test: fix snapshot test of macro expansion to a specific OS

    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 committed Jul 9, 2024
    Configuration menu
    Copy the full SHA
    5c2c21f View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    dcda90e View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    5ddd5e5 View commit details
    Browse the repository at this point in the history