Skip to content

LGP 4.0.0 and Gradle 9.0 test#603

Merged
siddhusaladi merged 13 commits into
OpenLiberty:mainfrom
siddhusaladi:LGP-4.x-test
Jun 26, 2026
Merged

LGP 4.0.0 and Gradle 9.0 test#603
siddhusaladi merged 13 commits into
OpenLiberty:mainfrom
siddhusaladi:LGP-4.x-test

Conversation

@siddhusaladi

Copy link
Copy Markdown
Contributor

#580
Added test for Liberty Gradle Plugin 4.0.0 and Gradle 9.0 support
Wrote closeWorkspace() function to enable standalone test execution (Also removed init test files as a result)

@siddhusaladi siddhusaladi requested a review from mattbsox June 18, 2026 20:47

@sharma1208 sharma1208 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Everything LGTM just one question about whether a sleep was necessary or not and whether init project test files should be deleted or not! Also ran the tests locally on grade and maven previous version and it was working, so Matt said it should be good.

Comment thread src/test/Gradle9TestDevModeActions.ts Outdated
sharma1208
sharma1208 previously approved these changes Jun 19, 2026

@sharma1208 sharma1208 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yay! Ask Matt if he wants to review it still but looks good!

mattbsox
mattbsox previously approved these changes Jun 22, 2026

@mattbsox mattbsox left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me. Let's see if the build issues resolve themselves with a newer VSCode version though.

@siddhusaladi siddhusaladi dismissed stale reviews from mattbsox and sharma1208 via 3bc9d69 June 23, 2026 16:43
@siddhusaladi siddhusaladi requested a review from sharma1208 June 26, 2026 18:36

@sharma1208 sharma1208 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good just fix the step number! It might be present other places too in this file so just double check

Comment thread src/test/Gradle9TestDevModeActions.ts

@sharma1208 sharma1208 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks good to me just double check if Matt wants to review! Also probably wait for the builds to run and make sure the only error is the rest-snippet test?

@siddhusaladi siddhusaladi merged commit 6ea292c into OpenLiberty:main Jun 26, 2026
0 of 8 checks passed
@siddhusaladi siddhusaladi deleted the LGP-4.x-test branch June 26, 2026 20:13
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.

3 participants