-
Notifications
You must be signed in to change notification settings - Fork 5
Bump upload/download artifact and configure dependabot to group them #161
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #161 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 2 2
Lines 16 16
=========================================
Hits 16 16 ☔ View full report in Codecov by Sentry. |
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 to me, couple thoughts but nothing meaningful
@@ -36,9 +39,9 @@ runs: | |||
shell: bash | |||
|
|||
- name: Upload lockfiles | |||
uses: actions/upload-artifact@v3 | |||
uses: actions/upload-artifact@v4 |
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.
Might be nice to fully qualify the version here
@@ -10,6 +10,9 @@ inputs: | |||
python_version: | |||
description: Python version to install | |||
default: "3.x" | |||
artifact_name: | |||
description: A user friendly name to give the produced artifacts | |||
default: "tests" |
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.
Might be better off without a default here 🤷
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.
Yh, I was only being lazy, I'll remove the default
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 to me.
It's required to bump both dependencies at the same time.
To test: