-
Notifications
You must be signed in to change notification settings - Fork 417
Start of using bloop rifle #678
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
Conversation
Should fix https://youtrack.jetbrains.com/issue/SCL-21024/Use-a-locally-installed-bloop-with-provided-connection-file#focus=Comments-27-11406726.0-0 * Remove old bloop launcher * Fix some warnings which became errors in the build, by adding @scala.annotation.nowarn("cat=deprecation") where needed. TODO: * Make the bloop version configurable.
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.
i dont understand why this file was edited at all.
|
Hello, I noticed that you used the new bloop rifle. What are the practical benefits of using it? |
|
The bloop-launcher is no longer maintained, and is replaced by bloop-rifle. |
- based on changes from #678
|
Hello, The fix will be available in the next 2025.2 nightly version. If you're not using nightly versions, you can enable them in the Thank you again for the contribution. |
|
Because this PR is merged, SCL-21024 is also marked as "fixed". At the bottom of SCL-21024, there is a note incorrectly indicating that the PR was declined. This happened because the cherry-pick wasn't done with your changes, although they are explicitly mentioned in the merged commit. |
|
Great news. Thanks a lot! |
- based on changes from #678
Should fix https://youtrack.jetbrains.com/issue/SCL-21024/Use-a-locally-installed-bloop-with-provided-connection-file#focus=Comments-27-11406726.0-0
TODO: