Conversation
Codecov ReportPatch coverage:
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more. Additional details and impacted files@@ Coverage Diff @@
## master #2508 +/- ##
==========================================
- Coverage 98.33% 98.32% -0.01%
==========================================
Files 130 131 +1
Lines 12978 13140 +162
==========================================
+ Hits 12762 12920 +158
- Misses 216 220 +4
☔ View full report in Codecov by Sentry. |
|
Can we write inline type annotation instead of seprated pyi for new file? |
cab4757 to
934681e
Compare
@trim21 This ended up being a bit more difficult trying to bridge the gap from existing .pyi structure. I believe I was able to get this implemented correctly, but had to make some updates to other .pyi file dependancies to pass linting across all python versions. Apologies if folks were getting emailed about all the CI builds while testing, I eventually realized I was getting the same mypy linting errors locally and was able to continue validation without triggering the CI |
EnricoMi
left a comment
There was a problem hiding this comment.
Excellent pull request, can we extend the tests please? I think the replay data are already sufficient, just more assertions.
|
@EnricoMi Added assertions for all attributes in Organization and Repository as well as adding the same replay/test workflow to |
08c0f35 to
acb8a62
Compare
5765c0b to
a2428df
Compare
a2428df to
16ffa39
Compare
|
@jmgreg31 sorry for the delay. We have reworked some related code, which broke your pull request. I have resolved the issues and changed |
github/PaginatedList.py
Outdated
|
|
||
|
|
||
| class PaginatedList(PaginatedListBase): | ||
| class PaginatedList(PaginatedListBase, Generic[T]): |
There was a problem hiding this comment.
this generic is not actually used in this pr?
Webhook Deliveries
Add support for Organization and Repository Webhook Deliveries
Summary of Changes
get_hook_deliveryandget_hook_deliveriesHookDeliverymodel wheregithub.HookDelivery.HookDeliveryis a subclass ofgithub.HookDelivery.HookDeliverySummaryfor DRY implementationResolves #2507