feat: support listing all branches#178
Conversation
| * @return list of 30 branches in repository | ||
| * @deprecated Doesn't return all branches. Use {@link RepositoryClient#listAllBranches} instead | ||
| */ | ||
| @Deprecated |
There was a problem hiding this comment.
Hi and thank you for your contribution! I don't think we really need to deprecate this method?
Did a similar PR recently: #176 without deprecating the non paged method. Otherwise I think this PR looks good!
There was a problem hiding this comment.
Somehow I think developers should be warned to not use the old method and point them in a better direction. Depreciation was the clearest way I could think of for doing that. But I agree that depreciation is a bit wrong in the sense that the old method isn't likely to be removed.
There was a problem hiding this comment.
As the public methods in this repo usually have javadocs, I think it might be nice to just mention the new (better) method in the javadocs without labelling the old method as deprecated..
There was a problem hiding this comment.
The good think with deprecation is that it gives a visible warning in peoples IDEs. But OK, how about this?
Also added convenience method for making a stream of paginated results
1febf45 to
b9d9f14
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #178 +/- ##
============================================
- Coverage 77.23% 77.17% -0.07%
- Complexity 308 311 +3
============================================
Files 43 44 +1
Lines 1028 1034 +6
Branches 44 44
============================================
+ Hits 794 798 +4
- Misses 209 211 +2
Partials 25 25 ☔ View full report in Codecov by Sentry. |
felix-seifert
left a comment
There was a problem hiding this comment.
This now sounds like a nice compromise to hint lib users to use the paged method instead!
Also added convenience method for making a stream of paginated results