You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Justification:
I build a desktop app using Compose Multiplatform, and the best distribution to use when building/packaging it is JetBrains Runtime because it expands the runtime support for GUI apps, expanding Swing and AWT support for every platform and fixing bugs. I use Github Actions to build my app so having it be supported in this action would be very helpful.
Are you willing to submit a PR?
Im not familair with TypeScript/JavaScript so I cannot say yes.
Reacted by Nic Bell, Samael, Azim Muradov, TarCV, Messias Junior, Eduardo Martínez, Sebastiano Poggi, Danil Ovchinnikov, Carlos Hwa, Milosz Kalucki and 8 more
Reacted by Manuel Carrasco Moñino, Azim Muradov, Messias Junior, Hussein Shammari, Irineu A. Silva and Apolo
Reacted by YOUNG HO CHA (aka ganachoco), Azim Muradov, Messias Junior, sikmi_watanabe, AmirHossein Abdolmotallebi, Hussein Shammari, sal0max and Suresh
hi @Syer10, after the consultation with the team we have decided we are not currently planning to implement this feature. But I will not close this issue, because we want to see how much attention it is going to get :)
hi @yeikel, sorry for the delayed response. We are still not going to close this feature request, but still keeping it open for same reasons that I mentioned in the comment above :)
@yeikel why not create a branch in your fork that adds the support and then link to it here. We'll be able to see how much your repository is used pretty easily (GitHub tracks dependencies).
@yeikel why not create a branch in your fork that adds the support and then link to it here. We'll be able to see how much your repository is used pretty easily (GitHub tracks dependencies).
My intention was to contribute to this project in general, and I saw an issue I could try
I have no particular interest or need to support this runtime, and maintaining a fork is not something I am interested at this time
There's no obligation to maintain your fork. This repository doesn't have high churn and you aren't obligated to support it.
For anyone curious, adding support for a distribution really doesn't look particularly hard, you basically implement two files installer.js and model.js and then add some references. Here's an example: 5ffc13f
Once you've gotten it working in your fork, you can advertise it here.
This is a little late, but I submitted a PR that adds support for this:
- name: Setup JBR 21uses: gmitch215/setup-java@6d2c5e1f82f180ae79f799f0ed6e3e5efb4e664dwith:
distribution: 'jetbrains'java-version: 21cache: 'gradle'# For GitHub API (use if you believe you'd run into rate limits)env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
Reacted by Josh Soref, Mitchell Syer, Suresh, Nic Bell, Messias Junior, Gregory Mitchell, Paul Merlin and Stephen Costeira
Reacted by Sebastiano Poggi, Azim Muradov, Messias Junior and Gregory Mitchell
Activity
[-]Support JetbrainsRuntime distribution[/-][+]Support JetBrainsRuntime distribution[/+]marko-zivic-93 commentedon Nov 4, 2022
Hello @Syer10
Thank you for your feature request. We will investigate this and come back to you as soon as possible :)
panticmilos commentedon Nov 14, 2022
hi @Syer10, after the consultation with the team we have decided we are not currently planning to implement this feature. But I will not close this issue, because we want to see how much attention it is going to get :)
yeikel commentedon Feb 16, 2023
@panticmilos Would you accept community contributions for this? If so, feel free to assign this to me
jsoref commentedon Mar 6, 2023
It doesn't seem like they'd accept a community contribution without first showing demand.
But you could make a fork that included it and the evidence of repositories using your fork would be a good way to convince them to integrate it.
panticmilos commentedon Mar 7, 2023
hi @yeikel, sorry for the delayed response. We are still not going to close this feature request, but still keeping it open for same reasons that I mentioned in the comment above :)
nicbell commentedon Apr 18, 2023
This sounds like a good idea especially for Android projects as it will match Android Studio.
SingingBush commentedon Jul 20, 2023
I'd certainly like setup-java to support jbr as a distribution. It's worth doing for people that produce plugins for Jetbrains IDE's.
jsoref commentedon Jul 20, 2023
@yeikel why not create a branch in your fork that adds the support and then link to it here. We'll be able to see how much your repository is used pretty easily (GitHub tracks dependencies).
yeikel commentedon Jul 20, 2023
My intention was to contribute to this project in general, and I saw an issue I could try
I have no particular interest or need to support this runtime, and maintaining a fork is not something I am interested at this time
jsoref commentedon Jul 20, 2023
🤷
There's no obligation to maintain your fork. This repository doesn't have high churn and you aren't obligated to support it.
For anyone curious, adding support for a distribution really doesn't look particularly hard, you basically implement two files
installer.jsandmodel.jsand then add some references. Here's an example:5ffc13f
Once you've gotten it working in your fork, you can advertise it here.
Once it's advertised, you can look at the equivalent of this to see how much your version is used:
https://github.com/actions/setup-java/network/dependents
For example:
https://github.com/jord1e/setup-java/network/dependents
MessiasLima commentedon Oct 26, 2023
Would be nice to have thin in GitHub actions
rock3r commentedon Jan 30, 2024
Would it be possible to re-consider this, especially now that FooJay discover APIs support the JetBrains Runtime? It's very useful for Compose Multiplatform projects such as Jewel.
gmitch215 commentedon Jun 28, 2024
This is a little late, but I submitted a PR that adds support for this:
ApoloApps commentedon Nov 20, 2024
any news on this?
gmitch215 commentedon Nov 20, 2024
See #637