Skip to content
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

24012 release prep #529

Merged
merged 4 commits into from
Dec 6, 2024
Merged

Conversation

awisniew90
Copy link
Contributor

No description provided.

Signed-off-by: Adam Wisniewski <[email protected]>
Signed-off-by: Adam Wisniewski <[email protected]>
Signed-off-by: Adam Wisniewski <[email protected]>
Signed-off-by: Adam Wisniewski <[email protected]>
Copy link
Member

@mezarin mezarin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Adam, the changes look good.
I had a couple of comments and here are a couple more since there are no file changes for this:

  • Update user-guide.xml: software requirements-> IDE->table section-> Add new support.
  • Update installation.md: Install New Software -> Liner 38 and 46.

<unit id="org.eclipse.swtbot.ide.feature.group" version="3.1.0.202106041005"/>
</location>
<location includeAllPlatforms="false" includeConfigurePhase="true" includeMode="planner" includeSource="true" type="InstallableUnit">
<repository location="https://download.eclipse.org/tools/cdt/releases/10.6"/>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have you thought about upgrading the CDT release version to https://download.eclipse.org/tools/cdt/releases/10.6. The current one is 2 years old.

Similarly: Consider updating this also to: https://download.eclipse.org/technology/swtbot/releases/4.2.1/. The one we are using is 3 years old.

These are just examples. But you should also consider updating lsp4mp, etc.

@@ -24,7 +24,7 @@
<parent>
<groupId>io.openliberty.tools.eclipse</groupId>
<artifactId>parent</artifactId>
<version>24.0.9-SNAPSHOT</version>
<version>24.0.12-SNAPSHOT</version>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update comment on line 110

@awisniew90
Copy link
Contributor Author

Thanks @mezarin - I'll make a note of the CDT and SWTBot items. I'm going to address those and the others comments in future PRs. Following through the list here: #528

For this PR, I mostly just want to bump up to the new version.

@awisniew90 awisniew90 merged commit 3a789d2 into OpenLiberty:main Dec 6, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants