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.

Sign up for GitHub

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

Jump to bottom

Made pyo3 optional. #144

Merged
merged 9 commits into from
Aug 3, 2021
Merged

Made pyo3 optional. #144

merged 9 commits into from
Aug 3, 2021

Commits on Aug 3, 2021

  1. Made pyo3 optional.

    @drobnyjt
    drobnyjt committed Aug 3, 2021
    Configuration menu
    Copy the full SHA
    1ff4c0c View commit details
    Browse the repository at this point in the history

  2. Updated setup.py to ensure pyo3 is built.

    @drobnyjt
    drobnyjt committed Aug 3, 2021
    Configuration menu
    Copy the full SHA
    8a4d96e View commit details
    Browse the repository at this point in the history

  3. Fixed missing comma.

    @drobnyjt
    drobnyjt committed Aug 3, 2021
    Configuration menu
    Copy the full SHA
    7848735 View commit details
    Browse the repository at this point in the history

  4. More fixes to setup.py - I was using the setuptools Rust API incorrec… 

    …tly, and it was silently failing.
    @drobnyjt
    drobnyjt committed Aug 3, 2021
    Configuration menu
    Copy the full SHA
    64bcac7 View commit details
    Browse the repository at this point in the history

  5. Add python binding installation to workflow 

    I've added a simple check to make sure the python bindings install correctly from scratch.
    @drobnyjt
    drobnyjt authored Aug 3, 2021
    Configuration menu
    Copy the full SHA
    35e3ea1 View commit details
    Browse the repository at this point in the history

  6. Fix to workflow; typo.

    @drobnyjt
    drobnyjt authored Aug 3, 2021
    Configuration menu
    Copy the full SHA
    7cf1af5 View commit details
    Browse the repository at this point in the history

  7. Ensure rust compiler is installed 

    Python was having trouble finding the rust compiler, so I've added that to the workflow.
    @drobnyjt
    drobnyjt authored Aug 3, 2021
    Configuration menu
    Copy the full SHA
    0943871 View commit details
    Browse the repository at this point in the history

  8. Add sudo 

    For some reason, cargo test is now throwing a permission denied error in the workflow - sudo added to see if that fixes it.
    @drobnyjt
    drobnyjt authored Aug 3, 2021
    Configuration menu
    Copy the full SHA
    8dbc9c5 View commit details
    Browse the repository at this point in the history

  9. More permissions issues

    @drobnyjt
    drobnyjt authored Aug 3, 2021
    Configuration menu
    Copy the full SHA
    fbdc452 View commit details
    Browse the repository at this point in the history