-
Notifications
You must be signed in to change notification settings - Fork 33
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
Add to new project and include PRs #366
Conversation
@@ -4,6 +4,11 @@ on: | |||
issues: | |||
types: | |||
- opened | |||
pull_request: | |||
branches: | |||
- main |
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.
Unsure whether we would want to include PRs targeting all branches... 🤔
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.
I think yes... if we ever need to do work on a released branch or something... deleting!
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #366 +/- ##
==========================================
+ Coverage 65.23% 65.26% +0.02%
==========================================
Files 35 35
Lines 3843 3843
==========================================
+ Hits 2507 2508 +1
+ Misses 1138 1136 -2
- Partials 198 199 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
|
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.
Nice, changes look good and should alleviate all the manual interaction
No description provided.