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

[python/ci] Remote-storage CI YAML #3675

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

johnkerl
Copy link
Member

@johnkerl johnkerl commented Feb 5, 2025

Issue and/or context: [sc-25441]

Changes:

We have some release-checks notebooks we've been using for a year or two now. The current PR simply migrates that pre-existing content to a context in which these checks can be run more often, and sooner, and more conveniently.

Notes for Reviewer:

There's more to do here (see the Shortcut link above) -- namely, more UDF tests -- but this PR has a big enough line-count already. I'll add more UDF tests on a subsequent PR.

As well, analogous R tests will be a separate PR.

@johnkerl johnkerl force-pushed the kerl/python-remote-storage-ci branch 30 times, most recently from 866a5f6 to 8725ec9 Compare February 6, 2025 18:59
@johnkerl johnkerl force-pushed the kerl/python-remote-storage-ci branch from 5aa7ffc to a641eec Compare February 6, 2025 22:07
@johnkerl johnkerl force-pushed the kerl/python-remote-storage-ci branch 6 times, most recently from 8b59975 to 4d7a75b Compare February 7, 2025 15:38
@johnkerl johnkerl force-pushed the kerl/python-remote-storage-ci branch from 4d7a75b to 10e5d87 Compare February 7, 2025 16:10
@johnkerl johnkerl force-pushed the kerl/python-remote-storage-ci branch 3 times, most recently from a9ad909 to fac2879 Compare February 7, 2025 18:24
@johnkerl johnkerl requested review from nguyenv and jp-dark February 7, 2025 18:26
@johnkerl johnkerl marked this pull request as ready for review February 7, 2025 18:26
@johnkerl johnkerl force-pushed the kerl/python-remote-storage-ci branch 3 times, most recently from 6559ead to 57f08e8 Compare February 7, 2025 18:29
@johnkerl johnkerl changed the title [python/ci] Remote-storage CI YAML [WIP] [python/ci] Remote-storage CI YAML Feb 7, 2025
@johnkerl johnkerl force-pushed the kerl/python-remote-storage-ci branch from 57f08e8 to 4030d74 Compare February 7, 2025 18:37
@johnkerl johnkerl force-pushed the kerl/python-remote-storage-ci branch from deb672a to 873c545 Compare February 7, 2025 18:50
Copy link

codecov bot commented Feb 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.33%. Comparing base (2105abc) to head (873c545).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3675      +/-   ##
==========================================
+ Coverage   86.29%   86.33%   +0.04%     
==========================================
  Files          55       55              
  Lines        6390     6390              
==========================================
+ Hits         5514     5517       +3     
+ Misses        876      873       -3     
Flag Coverage Δ
python 86.33% <ø> (+0.04%) ⬆️

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

Components Coverage Δ
python_api 86.33% <ø> (+0.04%) ⬆️
libtiledbsoma ∅ <ø> (∅)

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.

1 participant