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

DM-35310: Modernize version determination and package setup #111

Merged
merged 8 commits into from Jun 23, 2022

Conversation

timj
Copy link
Member

@timj timj commented Jun 23, 2022

Checklist

  • ran Jenkins
  • added a release note for user-visible changes to doc/changes

@codecov
Copy link

codecov bot commented Jun 23, 2022

Codecov Report

Merging #111 (b64f9d7) into main (78985d2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #111   +/-   ##
=======================================
  Coverage   72.94%   72.94%           
=======================================
  Files          37       37           
  Lines        2506     2506           
  Branches      437      437           
=======================================
  Hits         1828     1828           
  Misses        609      609           
  Partials       69       69           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 78985d2...b64f9d7. Read the comment docs.

@timj
Copy link
Member Author

timj commented Jun 23, 2022

@mxk62 well the good news is that this PR passes the tests so lsst/ctrl_bps_panda#10 is still a bit of a mystery.

@timj timj requested a review from mxk62 June 23, 2022 21:34
@timj timj merged commit daf8b17 into main Jun 23, 2022
@timj timj deleted the tickets/DM-35310 branch June 23, 2022 21:43
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

2 participants