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

build: updates spun off mastiff PR #2723

Merged
merged 2 commits into from
Aug 19, 2023
Merged

Conversation

luizirber
Copy link
Member

@luizirber luizirber commented Aug 19, 2023

(There are too many things happening in #2230, so I'll open a couple of PRs to simplify it)

  • Bring tox config as close as possible to tox 4, but keep using tox 3 until tox-conda is compatible with tox 4. Ended up adding an explicit 'tox>=3.27,<4' to the conda dev env instructions to get the right version, and remove/replace deprecated options from tox.ini
  • Bump MACOSX_DEPLOYMENT_TARGET to 11.0 consistently for maturin and CI.
  • Remove Python installation for Rust tests in CI (not used)
  • Clean up flake.nix and avoid naersk, prefer regular nixpkgs tooling now that python3Packages.sourmash: init at 4.8.3 NixOS/nixpkgs#230596 is merged and we can keep them in sync
  • Move .ci/install_cargo.sh into cibuildwheel configuration, set up extra tools needed for bindgen/cbindgen (libclang and env vars)

@luizirber luizirber force-pushed the lirber/202308_build_updates branch from 23205d5 to 41303aa Compare August 19, 2023 22:26
@codecov
Copy link

codecov bot commented Aug 19, 2023

Codecov Report

Merging #2723 (75c0eff) into latest (9f6664e) will not change coverage.
The diff coverage is n/a.

❗ Current head 75c0eff differs from pull request most recent head c1a7825. Consider uploading reports for the commit c1a7825 to get more accurate results

@@           Coverage Diff           @@
##           latest    #2723   +/-   ##
=======================================
  Coverage   85.27%   85.27%           
=======================================
  Files         133      133           
  Lines       15191    15191           
  Branches     2614     2614           
=======================================
  Hits        12954    12954           
  Misses       1936     1936           
  Partials      301      301           
Flag Coverage Δ
hypothesis-py 25.80% <ø> (ø)
python 92.83% <ø> (ø)
rust 51.64% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@luizirber luizirber force-pushed the lirber/202308_build_updates branch 2 times, most recently from 7a9cfa8 to 1c24f6c Compare August 19, 2023 22:53
@luizirber luizirber force-pushed the lirber/202308_build_updates branch from 1c24f6c to 75c0eff Compare August 19, 2023 23:05
@luizirber luizirber requested a review from a team August 19, 2023 23:17
@luizirber
Copy link
Member Author

Ready for review and merge @sourmash-bio/devs

@luizirber luizirber force-pushed the lirber/202308_build_updates branch from 75c0eff to c1a7825 Compare August 19, 2023 23:34
@luizirber luizirber merged commit 0dd0e3b into latest Aug 19, 2023
@luizirber luizirber deleted the lirber/202308_build_updates branch August 19, 2023 23:37
@ctb
Copy link
Contributor

ctb commented Aug 21, 2023

Note, this also adds cargo denyto flake.nix. Are we using that for anything, @luizirber ?

Update: ahh, had not gotten to #2724 yet.

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.

2 participants