Skip to content

Migrate hazelcast-wm to new Maven Central [DI-630]#210

Merged
nishaatr merged 11 commits intomasterfrom
wm-new-portal
Oct 2, 2025
Merged

Migrate hazelcast-wm to new Maven Central [DI-630]#210
nishaatr merged 11 commits intomasterfrom
wm-new-portal

Conversation

@nishaatr
Copy link
Contributor

@nishaatr nishaatr commented Sep 30, 2025

Changes

  • This is OSS only project
  • Found Findbugs issues and resolved separately Fix Findbugs errors [DI-630] #212
  • Release deployed via plugin but SNAPSHOT via <distributionManagement> as using separate profiles
  • fixed <developer>
  • added MAVEN_ARGS in GH workflows
  • updated Maven wrapper version in .mvn/wrapper/maven-wrapper.properties (copied from Mono); otherwise, MAVEN_ARGS doesn't take effect
  • removed unnecessary -Dgpg.passphrase when calling Maven in worflows as we have MAVEN_GPG_PASSPHRASE
  • updated workflows to use local actions/setup-jdk

Testing

Snapshot

Version: 5.2-SNAPSHOT

Release

Version: 1.0.0

Made changes on temp branch diff

  1. Ran build successfully
  2. Validated deployment available for publishing here
  3. image

Post tasks

  • Remove staged deployment
  • Remove temp branches

Fixes DI-630

@github-actions
Copy link

github-actions bot commented Sep 30, 2025

Test Results

 15 files   15 suites   24m 57s ⏱️
186 tests 186 ✅ 0 💤 0 ❌
331 runs  331 ✅ 0 💤 0 ❌

Results for commit e7fe0d3.

♻️ This comment has been updated with latest results.

nishaatr added a commit that referenced this pull request Oct 1, 2025
`Findbugs` issues found during
#210

- fix
[errors](https://github.com/hazelcast/hazelcast-wm/actions/runs/18138331537/job/51622684517#step:8:395)
using `volatile`
- disabled Jenkins
[WM-pr-builder](https://jenkins.hazelcast.com/job/WM-pr-builder/) as we
have GH one. I checked and mirrored config. Tomasz thinks Josef might
have started the migration but didn't have a chance to complete
Copy link
Contributor

Choose a reason for hiding this comment

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

We should upgrade this separately and it should be managed automatically. I will take a look.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks
revert or keep?
worth checking hazelcast-jdbc also

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sorry missed the approval so will keep

Copy link
Contributor

Choose a reason for hiding this comment

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

thanks revert or keep? worth checking hazelcast-jdbc also

I've already done hazelcast-jdbc. Not sure how this was missed when I first introduced it.
I'd say revert to make this PR smaller but I don't really mind.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

great.
prefer to keep.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

git fetch --all
git merge origin/master --strategy-option theirs

6399716

Copy link
Contributor

Choose a reason for hiding this comment

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

You could just've done it from the GitHub web UI...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

didn't come to mind.
preferred manual just in case!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

but GH UI still showing Update branch 😞
looks like known behavior
says change base branch temporarily via PR edit.
I will skip but good to know!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ignore. just updated. master was truly updated #217

@nishaatr nishaatr merged commit 6726be7 into master Oct 2, 2025
4 checks passed
@nishaatr nishaatr deleted the wm-new-portal branch October 2, 2025 14:07
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