-
Notifications
You must be signed in to change notification settings - Fork 36
Conversation
Is the lock file supposed to be included? |
yes |
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 right (havn't tested locally)
Co-authored-by: trag1c <[email protected]> Co-authored-by: trag1c <[email protected]>
Co-authored-by: trag1c <[email protected]>
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.
The coverage action is failing due to missing pytest
command, and the
Codecov Report
@@ Coverage Diff @@
## main #468 +/- ##
==========================================
+ Coverage 64.98% 65.01% +0.03%
==========================================
Files 184 183 -1
Lines 5971 5968 -3
==========================================
Hits 3880 3880
+ Misses 2091 2088 -3 Continue to review full report at Codecov.
|
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.
lgtm
All changes are implemented, all checks pass. 2 approvals. Merging now |
Changes
Pincer now used Poetry instead of a mix of pip and Pipfile. This will make it easier to publish and removes the hassle.
Check off the following