Skip to content

Conversation

mkarimi23
Copy link
Contributor

Check ID type is changed from int to long in this PR

This PR changes the method parameter for updateCheckRun from int to long to follow the changes and avoid lossy conversions in method invocations.

@codecov
Copy link

codecov bot commented Dec 12, 2022

Codecov Report

Merging #120 (00f9bcf) into master (9a4b09f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #120   +/-   ##
=========================================
  Coverage     73.59%   73.59%           
  Complexity      253      253           
=========================================
  Files            40       40           
  Lines           890      890           
  Branches         40       40           
=========================================
  Hits            655      655           
  Misses          210      210           
  Partials         25       25           
Impacted Files Coverage Δ
...va/com/spotify/github/v3/clients/ChecksClient.java 90.90% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@mkarimi23 mkarimi23 merged commit dbb90ff into master Dec 12, 2022
@mkarimi23 mkarimi23 deleted the update_checkrun_id_type branch December 12, 2022 14:30
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