Skip to content

use GraphQL API to get issue counts from GitHub#1229

Merged
haarg merged 1 commit into
masterfrom
haarg/github-graphql
Jul 7, 2024
Merged

use GraphQL API to get issue counts from GitHub#1229
haarg merged 1 commit into
masterfrom
haarg/github-graphql

Conversation

@haarg
Copy link
Copy Markdown
Member

@haarg haarg commented Jun 19, 2024

GitHub's GraphQL API can return the number of issues directly, rather than needing to retrieve and count them.

Net::GitHub::V4 provides access to the GitHub GraphQL API. Switch to using it rather than Pithub.

mickeyn
mickeyn previously approved these changes Jun 19, 2024
GitHub's GraphQL API can return the number of issues directly, rather
than needing to retrieve and count them.

Net::GitHub::V4 provides access to the GitHub GraphQL API. Switch to
using it rather than Pithub.
@haarg haarg force-pushed the haarg/github-graphql branch from b272bc7 to b96ee0c Compare June 20, 2024 17:07
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 20, 2024

Codecov Report

Attention: Patch coverage is 25.00000% with 6 lines in your changes missing coverage. Please review.

Project coverage is 60.81%. Comparing base (a3834f6) to head (b96ee0c).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1229      +/-   ##
==========================================
+ Coverage   60.74%   60.81%   +0.06%     
==========================================
  Files         163      163              
  Lines        4456     4453       -3     
  Branches      649      646       -3     
==========================================
+ Hits         2707     2708       +1     
+ Misses       1516     1512       -4     
  Partials      233      233              
Files Coverage Δ
lib/MetaCPAN/Script/Tickets.pm 78.37% <25.00%> (+4.35%) ⬆️

@haarg haarg merged commit eb8a5f0 into master Jul 7, 2024
@haarg haarg deleted the haarg/github-graphql branch July 7, 2024 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants