-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 support for JetBrains EAP IDEs (2022.3) #13400
Conversation
@@ -141,7 +137,7 @@ class GitpodClientProjectSessionTracker( | |||
override fun onNext(ps: Status.PortsStatusResponse) { | |||
for (port in ps.portsList) { | |||
// Avoiding undesired notifications | |||
if (ignorePorts.contains(port.localPort)) { | |||
if (ignoredPortsForNotificationService.getIgnoredPorts().contains(port.localPort)) { |
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.
The ignore ports functionality seems to be overcomplicated now, instead of being 3 lines of code, we have 3 files. Am I missing any benefit from doing so?
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.
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.
oh, I see - thanks!
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.
Werft is failing to create the Preview Environments for this PR, so I added instructions for testing it from the Production environment in the PR Description. |
/werft run 👍 started the job as gitpod-build-felladrin-jetbrains-ides-2022.3.7 |
ℹ️ I'm going to rebase this PR on the latest commit from the main branch. |
d485e9c
to
83d939a
Compare
@felladrin if Werft keeps failing, I suggest create a new branch and move the PR to a new one (with new prev env) |
Something is wrong with the connection between Gitpod Build and and this Github PR, cause the latest build is working fine: Note: I've unchecked the |
@felladrin I've followed the manual steps and it works as expected. Using I think you need an admin now to merge this PR given the failed Werft check. |
/werft run 👍 started the job as gitpod-build-felladrin-jetbrains-ides-2022.3.13 |
/unhold |
Description
Now that IDEA/PyCharm/PhpStorm/GoLand latest version is v2022.3 in Gitpod, we can add support for JetBrains IDEs v2022.3.
All code changes were done to accommodate the API differences between v2022.2 and v2022.3. There were no logic/flow changes in the plugin.
The dev server script (
launch-dev-server.sh
) has been improved to allow us locally test EAP and Stable versions of the IDE, which is a common point of failure during Werft builds - because there we build the plugin for EAP and Stable IDE versions simultaneously.- By default, running
launch-dev-server.sh
, will launch the JetBrains EAP version.- If we want to build launch JetBrains Stable version we run it with the additional 's' parameter:
launch-dev-server.sh -s
How to test
1. Access the Preview Environment of this PR2. Open a workspace using the Stable Version of IntelliJ IDEA and check if the Control Center is being displayed correctly and that the IDE shows no errors about Gitpod Plugin.
3. Do the same as Step 2, but now using the Latest Stable Version of IntelliJ IDEA.
We won't be able to test on preview environments, as the EAP JetBrains IDEs versions are pinned to 2022.2.1 during the Self-Hosted release. So please test it on gitpod.io by following these steps:
cd components/ide/jetbrains/backend-plugin/
./launch-dev-server.sh -s
to launch the Stable Version of IntelliJ IDEA and check if the Control Center is being displayed correctly and that the IDE shows no errors about Gitpod Plugin.rm -rf /workspace/ide-backend
./launch-dev-server.sh
to launch the Latest Stable Version of IntelliJ IDEA, and do the same checks from Step 3.Release Notes
Werft options:
If enabled this will build
install/preview
Valid options are
all
,workspace
,webapp
,ide