Skip to content

feat: Add tasks for use in git#2

Merged
effigies merged 6 commits into
mainfrom
rf/dashboard-utils
Mar 19, 2026
Merged

feat: Add tasks for use in git#2
effigies merged 6 commits into
mainfrom
rf/dashboard-utils

Conversation

@effigies
Copy link
Copy Markdown
Contributor

This was part of an experiment with refactoring two codebases simultaneously. The goal was to reduce the duplication of code between https://github.com/OpenNeuroOrg/dashboard scripts and this package.

By adding/adjusting these three functions here, essentially all of the duplication was removed from dashboard. We can keep that repository focused on the schema.

Rename LatestSnapshot to Snapshot (with backward-compatible alias),
add optional snapshots field to DatasetNode, add
GET_DATASETS_WITH_SNAPSHOTS query, and add include_snapshots
keyword parameter to get_page().
Add list_refs() that runs git ls-remote --symref and returns a
structured map of branches, tags, and HEAD.
Wraps list_s3_objects_pages() to collect paginated results
into a single set of keys.
Comment thread src/ondiagnostics/graphql.py Outdated
Comment thread tests/test_tasks_s3.py Outdated
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 19, 2026

Codecov Report

❌ Patch coverage is 99.25926% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 97.85%. Comparing base (3fcb645) to head (f8e4f53).
⚠️ Report is 7 commits behind head on main.

Files with missing lines Patch % Lines
src/ondiagnostics/tasks/git.py 97.36% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #2      +/-   ##
==========================================
+ Coverage   97.69%   97.85%   +0.16%     
==========================================
  Files          17       17              
  Lines        1128     1260     +132     
==========================================
+ Hits         1102     1233     +131     
- Misses         26       27       +1     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

effigies and others added 3 commits March 19, 2026 15:32
Technically breaking, but that's real nit-picky.

Co-authored-by: Chris Markiewicz <effigies@gmail.com>
@effigies effigies force-pushed the rf/dashboard-utils branch from 37c4b5f to f8e4f53 Compare March 19, 2026 19:35
@effigies effigies merged commit 68cf92e into main Mar 19, 2026
5 checks passed
@effigies effigies deleted the rf/dashboard-utils branch March 19, 2026 19:37
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