-
Notifications
You must be signed in to change notification settings - Fork 22
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
feat: Create New usePullsTeam Hook #2319
feat: Create New usePullsTeam Hook #2319
Conversation
✅ Deploy Preview for gazebo-staging ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov Report
@@ Coverage Diff @@
## main #2319 +/- ##
=======================================
- Coverage 33.91 33.80 -0.11
=======================================
Files 660 661 +1
Lines 8269 8294 +25
Branches 2070 2025 -45
=======================================
- Hits 2804 2803 -1
- Misses 5436 5469 +33
+ Partials 29 22 -7
... and 8 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #2319 +/- ##
==========================================
+ Coverage 33.90% 33.92% +0.01%
==========================================
Files 660 658 -2
Lines 8269 8227 -42
Branches 2052 2005 -47
==========================================
- Hits 2804 2791 -13
+ Misses 5436 5414 -22
+ Partials 29 22 -7
... and 17 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
Codecov Report
@@ Coverage Diff @@
## main #2319 +/- ##
==========================================
- Coverage 33.90% 33.79% -0.12%
==========================================
Files 660 661 +1
Lines 8269 8294 +25
Branches 2070 2069 -1
==========================================
- Hits 2804 2803 -1
- Misses 5436 5469 +33
+ Partials 29 22 -7
... and 8 files with indirect coverage changes Continue to review 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.
Lgtm, mainly checked that head.totals.percentCovered
+ changeCoverage
weren't present as this was a mirror of other code 👌
Codecov Report
@@ Coverage Diff @@
## main #2319 +/- ##
=======================================
Coverage ? 33.79%
=======================================
Files ? 661
Lines ? 8294
Branches ? 2025
=======================================
Hits ? 2803
Misses ? 5469
Partials ? 22
Continue to review full report in Codecov by Sentry.
|
Description
This PR creates a new hook that fetches a list of pulls with only the information required for the new team plan.
Closes codecov/engineering-team#679
Notable Changes
usePullsTeam
hook