-
-
Notifications
You must be signed in to change notification settings - Fork 70
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
Update GitHub workflow files #1224
Conversation
…r latest version of GitHub actions
Update a few GitHub actions' versions, and add dependabot to check fo…
Update SonarScan for .NET
Thanks a lot!! Love these kinds of contributions. I will do a proper review tomorrow after work. |
No rush. It all comes down to getting rid of warnings when building. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1224 +/- ##
====================================
Coverage 82% 82%
====================================
Files 196 196
Lines 3840 3840
Branches 426 426
====================================
Hits 3159 3159
Misses 516 516
Partials 165 165
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ 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, thanks for this!
@bakgaard please make sure the commits are signed for better security. I let this one pass for now since it was an easy review. |
Odd, I thought it was. |
* Update a few GitHub actions' versions, and add dependabot to check for latest version of GitHub actions * Update SonarScan for .NET
Breaking change
No breaking changes
Proposed change
ci_analyze.yml
when merging into maindependabot
Type of change
Additional information
Checklist
If user exposed functionality or configuration are added/changed: