Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1474 +/- ##
============================================
- Coverage 73.11% 73.06% -0.05%
+ Complexity 4382 4381 -1
============================================
Files 475 475
Lines 14244 14244
Branches 1447 1447
============================================
- Hits 10415 10408 -7
- Misses 2970 2977 +7
Partials 859 859 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
zimeg
left a comment
There was a problem hiding this comment.
💌 Notes for the most wonderful reviewers and readers alike!
| permissions: | ||
| contents: read |
There was a problem hiding this comment.
📣 Permissions for the build step are proving alright with just read! This is interesting to me, but makes sense in the context of uploading artifacts to action outputs perhaps?
📚 https://github.com/slackapi/java-slack-sdk/actions/runs/15126115805?pr=1474
| permissions: | ||
| contents: read | ||
| steps: | ||
| - uses: actions/checkout@v4 |
There was a problem hiding this comment.
🗣️ Changes unrelated to hashing steps are due to indentation! No change to testing or functionalities otherwise.
| permissions: | ||
| contents: read |
|
@WilliamBergamin Thank you for reviewing this! I am optimistic that CI will continue to remain stable here 🙏 ✨ |
Summary
This PR uses the wonderful
zizmortool to audit our own workflows andpinactfor pinned versioning 👾Category
Reviewers
A similar audit can be performed with the
zizmortool:Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you agree to those rules.