-
Notifications
You must be signed in to change notification settings - Fork 737
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add ability to search repos by fork and visibility #1188
Add ability to search repos by fork and visibility #1188
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1188 +/- ##
============================================
+ Coverage 74.16% 74.25% +0.09%
- Complexity 1874 1880 +6
============================================
Files 188 188
Lines 6182 6200 +18
Branches 368 368
============================================
+ Hits 4585 4604 +19
+ Misses 1375 1374 -1
Partials 222 222
Continue to review full report at Codecov.
|
c2ae62e
to
622017a
Compare
…|private|internal)
622017a
to
2c323d2
Compare
src/main/java/org/kohsuke/github/GHRepositorySearchBuilder.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Minor tweaks and questions. I'm open to discussion if you disagree with my suggestions.
…HRepositorySearchBuilder
Hi @bitwiseman |
@t0m4uk1991 If you disagree with the changes, please feel free to modify or revert them and we can discuss further. |
d7a19aa
to
74b1977
Compare
Added javadoc for each value of Fork search mode enum. Made passing UNKNOWN to visibility() throw a GHException. UNKNOWN a placeholder and should never be passed into any method in the API.
74b1977
to
1994f47
Compare
@bitwiseman |
Description
Updated GHRepositorySearchBuilder.java, add ability to search repos by fork:(true|only) and visibility:(public|private|internal)
This one issue #1174 should be closed after merge.
Fixes #1174