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
added LightGBM SWIG wrappers for macOS and updated docs #2002
Conversation
9b15372
to
06775c4
Compare
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.
Please fix rst headers structure.
LGTM! Waiting for Apple Clang test. |
@StrikerRUS I verified that this works with Apple Clang instructions. Are there any other pending issues that need to be fixed? |
@imatiach-msft Glad to hear such great news! Thank you very much! Unfortunately, I have no permission to merge a PR with any failed check. Could you please wait till #2016 will be merged and update the branch (I'll notify you). |
@imatiach-msft We've repaired CI checks. Please update the branch. |
d4edd02
to
44b2c8e
Compare
@StrikerRUS done! |
@imatiach-msft Awesome! Thank you very much! I think this PR needs a second review before merging. Ping @chivee @guolinke |
This should resolve Support for building Java Wrapper in OS X #1326