[JetBrains] Migrate backend-plugin platform version for 2024.2
#19978
Conversation
- Update Platform Version of JetBrains Backend Plugin (EAP) to 242.18071-EAP-CANDIDATE-SNAPSHOT
This reverts commit 4574632.
mustard-mh
changed the title
2024.2
| @@ -2,7 +2,7 @@ | |||
| // Licensed under the GNU Affero General Public License (AGPL). | |||
| // See License.AGPL.txt in the project root for license information. | |||
|
|
|||
| rootProject.name = "jetbrains-backend-plugin" | |||
| rootProject.name = "gitpod-remote" | |||
There was a problem hiding this comment.
We need to change the name here because new platform plugin will treat it as build result name
| @@ -1,5 +1,5 @@ | |||
| distributionBase=GRADLE_USER_HOME | |||
| distributionPath=wrapper/dists | |||
| distributionUrl=https\://services.gradle.org/distributions/gradle-7.5.1-bin.zip | |||
| distributionUrl=https\://services.gradle.org/distributions/gradle-8.8-bin.zip | |||
There was a problem hiding this comment.
This is required for new platform plugin (beta)
| - - "bash" | ||
| - "-c" | ||
| - > | ||
| echo java=21.0.3.fx-zulu > .sdkmanrc |
There was a problem hiding this comment.
Is it requirement only for build time? at runtime any java version will work still with 2024.2?
There was a problem hiding this comment.
2024.2 requires java 21
There was a problem hiding this comment.
ok, just asking that we don't need to change anything in our images, java 21 then is bundled together with IJ?
| // Licensed under the GNU Affero General Public License (AGPL). | ||
| // See License.AGPL.txt in the project root for license information. | ||
|
|
||
| import io.gitlab.arturbosch.detekt.Detekt |
There was a problem hiding this comment.
I don't think we need to duplicate this file.
We can use build.gradle.kts for latest, and build.gradle-stable.kts.
If someone wants to change they can unload one from IJ and load another.
for building we can pass different with ./gradlew --build-file
There was a problem hiding this comment.
Build failed with this change 3cc4494
Searched error Build cache controller already set that it may need --no-build-cache, but after modification, build failed again (if we copy it into build.gradle.kts, it will work):
There was a problem hiding this comment.
I'm going to revert that commit
There was a problem hiding this comment.
Base on the reverting, keep different build.gradle-<version>.kts is more useful for me: so that I could keep these two copies and use build.gradle.kts to debug with launcher-dev-server.sh
This reverts commit 3cc4494.
|
Integration tests (for stable) are passed Smoke tested stable IntelliJ IDEA, WebStorm and latest IDEA (terminal broken)
|
is it known and going to be fixed? |
It's not known, I need to ask JetBrains what happends (I see it found no BackendTerminalManager class in idea.log) As it's EAP versions, we could let it go and make terminal stuff another follow up issue |
|
@akosyakov for message above☝️ |
Description
Use new beta IntelliJ Platform Plugin which is required by JetBrains IDEs version
2024.2.*Related Issue(s)
Fixes ENT-350, ENT-330
How to test
(nit) Port forwarding
Backend Status Detailspage (click right top Gitpod icon) >Portstab, port8080should display correct state and functions like open in browser should workDocumentation
Preview status
Gitpod was successfully deployed to your preview environment.
Build Options
Build
Run the build with werft instead of GHA
Run Leeway with
--dont-testPublish
Installer
Add desired feature flags to the end of the line above, space separated
Preview Environment / Integration Tests
If enabled this will build
install/previewIf enabled this will create the environment on GCE infra
Saves cost. Untick this only if you're really sure you need a non-preemtible machine.
Valid options are
all,workspace,webapp,ide,jetbrains,vscode,ssh. If enabled,with-previewandwith-large-vmwill be enabled./hold