Skip to content

Conversation

@dennisgranath
Copy link
Contributor

@dennisgranath dennisgranath commented Mar 18, 2024

  • This adds a method for posting requests to the graphql endpoint.
  • It also adds the nodeId field to the PullRequest model (this is needed to be able to reference PR:s in graphql mutations)

@codecov
Copy link

codecov bot commented Mar 18, 2024

Codecov Report

Attention: Patch coverage is 31.03448% with 20 lines in your changes are missing coverage. Please review.

Project coverage is 76.15%. Comparing base (b8e8ae6) to head (16cb788).

Files Patch % Lines
...va/com/spotify/github/v3/clients/GitHubClient.java 31.03% 20 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #182      +/-   ##
============================================
- Coverage     77.28%   76.15%   -1.13%     
- Complexity      312      313       +1     
============================================
  Files            44       44              
  Lines          1039     1057      +18     
  Branches         44       44              
============================================
+ Hits            803      805       +2     
- Misses          211      227      +16     
  Partials         25       25              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dennisgranath dennisgranath marked this pull request as ready for review March 19, 2024 09:08
@dennisgranath dennisgranath changed the title WIP: enable automerge Add functionality for enabling automerge for a PR Mar 19, 2024
@dennisgranath dennisgranath changed the title Add functionality for enabling automerge for a PR Add basic functionality for running graphql requests Mar 19, 2024
Co-authored-by: Abhishek Jain <Abhi347@users.noreply.github.com>
@dennisgranath dennisgranath merged commit 38ee9cf into master Mar 19, 2024
@dennisgranath dennisgranath deleted the enableAutoMerge branch March 19, 2024 13:39
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.

3 participants