Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Add isArchived method handling archived property of Repository#85

Merged
Abhi347 merged 5 commits intospotify:masterspotify/github-java-client:masterfrom
vchernodon:vchernodon-patch-1vchernodon/github-java-client:vchernodon-patch-1Copy head branch name to clipboard
Apr 4, 2022
Merged

Add isArchived method handling archived property of Repository#85
Abhi347 merged 5 commits intospotify:masterspotify/github-java-client:masterfrom
vchernodon:vchernodon-patch-1vchernodon/github-java-client:vchernodon-patch-1Copy head branch name to clipboard

Conversation

@vchernodon
Copy link
Contributor

The archived property included and exposed via isArchived method.
This helps avoiding to make any github calls which would result in error when the repository is archived.
So before making such calls (create comment/status etc.) - the isArchived method could be checked to avoid making other calls.

@codecov
Copy link

codecov bot commented Mar 28, 2022

Codecov Report

Merging #85 (c029a8b) into master (b1b4cf2) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #85   +/-   ##
=========================================
  Coverage     70.20%   70.20%           
  Complexity      222      222           
=========================================
  Files            36       36           
  Lines           866      866           
  Branches         36       36           
=========================================
  Hits            608      608           
  Misses          233      233           
  Partials         25       25           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e7837c3...c029a8b. Read the comment docs.

@Abhi347 Abhi347 merged commit 1155a5b into spotify:master Apr 4, 2022
@vchernodon vchernodon deleted the vchernodon-patch-1 branch April 4, 2022 11:36
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.