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

Exclude artifacts from sdist and wheel #1902

Merged
merged 5 commits into from May 22, 2024
Merged

Conversation

penguinpee
Copy link
Contributor

Building the wheel from the PyPI sdist will include those files and
directories otherwise. They are also distributed in the uploaded wheel.

Building the wheel from the PyPI sdist will include those files and
directories otherwise. They are also distributed in the uploaded wheel.
@stephprince stephprince assigned stephprince and rly and unassigned stephprince May 13, 2024
Copy link

codecov bot commented May 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.93%. Comparing base (c7ff5ef) to head (19aa88a).

Additional details and impacted files
@@           Coverage Diff           @@
##              dev    #1902   +/-   ##
=======================================
  Coverage   83.93%   83.93%           
=======================================
  Files          27       27           
  Lines        2620     2620           
  Branches      685      685           
=======================================
  Hits         2199     2199           
  Misses        328      328           
  Partials       93       93           
Flag Coverage Δ
unit 83.93% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rly rly added this to the 2.8.0 milestone May 22, 2024
@rly
Copy link
Contributor

rly commented May 22, 2024

Thanks for the report @penguinpee , I'll take a look at this before the next release.

@rly rly requested a review from mavaylon1 May 22, 2024 18:47
@rly
Copy link
Contributor

rly commented May 22, 2024

Some tests are failing due to unrelated issues related to the Zarr tutorial and a recent Zarr update.

@rly rly enabled auto-merge (squash) May 22, 2024 22:37
@rly rly merged commit 014b143 into NeurodataWithoutBorders:dev May 22, 2024
16 of 23 checks passed
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.

None yet

4 participants