Skip to content

Jackpot 31 based on NetBeans 31 - #50

Merged
ebarboni merged 1 commit into
apache:masterfrom
ebarboni:jacksoncoreupdate
Sep 9, 2026
Merged

ebarboni merged 1 commit into
apache:masterfrom
ebarboni:jacksoncoreupdate

Conversation

@ebarboni

@ebarboni ebarboni commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

trying to update to released version of NetBeans (using dlcdn for speed)

Jdk 21 is baseline, not able to get openjdk 26 so doing only 21 and 25

fixed notice.

fixed webapp/pom.xml for more modern lib

tested setup/java but the java home is something with temurin inside instead of openjdk something

@ebarboni
ebarboni requested review from lahodaj and mbien September 8, 2026 15:28

@lahodaj lahodaj 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. (Assuming tests pass.) Thanks!

@mbien mbien 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.

@ebarboni

ebarboni commented Sep 9, 2026

Copy link
Copy Markdown
Contributor Author

the maven / tools seems tested ok.

well I see some error but I have no clue it was before or not.
https://github.com/apache/netbeans-jackpot30/actions/runs/34244963365/job/102124484614?pr=50#step:6:1646

@mbien

mbien commented Sep 9, 2026

Copy link
Copy Markdown
Member

oh. the job doesn't have an action which checks for test failures and the junit task is probably run with failOnError=false. Since tests failed and everything is still green. I didn't check the logs before approving - sorry.

@lahodaj

lahodaj commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

The tests also verify that the testing framework in Jackpot works correctly, and do a nested JUnit run. And some of these nested JUnit runs fail intentionally. I believe this is the case here. So, I wouldn't worry about the log containing seemingly failing JUnit tests, as long as the job itself is not reporting failures. Thanks!

@ebarboni

ebarboni commented Sep 9, 2026

Copy link
Copy Markdown
Contributor Author

Ok lets merge

@ebarboni
ebarboni merged commit 82559c3 into apache:master Sep 9, 2026
1 check 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.

3 participants