-
Notifications
You must be signed in to change notification settings - Fork 2
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
docs: update submodules to add the PR template there too #428
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #428 +/- ##
=======================================
Coverage 73.49% 73.49%
=======================================
Files 43 43
Lines 2852 2852
Branches 468 468
=======================================
Hits 2096 2096
Misses 671 671
Partials 85 85 ☔ 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.
good idea - I was routinely adding this. Thanks @joanise !
@roedoejet Please also review Related PRs LOL. You'd think in the PR where I'm adding the question "Related PRs", I would have listed these... 🤷♂️ |
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
f4b4c61
to
57df25a
Compare
PR Goal?
Broadcast our PR template in all EV repos