-
Notifications
You must be signed in to change notification settings - Fork 12
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
DM-39915: Change use of butler.datastore now that it's deprecated #354
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #354 +/- ##
==========================================
+ Coverage 82.72% 82.76% +0.03%
==========================================
Files 66 66
Lines 7269 7268 -1
Branches 1418 1418
==========================================
+ Hits 6013 6015 +2
+ Misses 1007 1006 -1
+ Partials 249 247 -2
☔ View full report in Codecov by Sentry. |
c3d4a12
to
9ca6691
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, few minor comments.
b46df0a
to
0dcb051
Compare
@@ -0,0 +1,41 @@ | |||
name: "Check commits can be merged" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@andy-slac I think I have something working now. Can you take a quick look at this action and review it please?
1a98be9
to
8704071
Compare
* Checks summary for DO NOT MERGE * Checks requirements.txt for ticket branches
Relies on lsst/daf_butler#862
Checklist
doc/changes