Clean up dependency management

[aloubyansky]

Add the deployment artifact to the BOM, remove imports of Quarkus project internal internal POMs (they don't seem to be useful anyway).

[devOpsHazelcast]

All committers have signed the CLA.

[aloubyansky]

This looks suspicious https://github.com/hazelcast/quarkus-hazelcast-client/blob/master/runtime/pom.xml#L56 Is there a reason it's not in the BOM? If it represents a runtime classpath requirement it should be added to the BOM.

[pivovarit]

Thanks! Super helpful! Was it supposed to be done like that since the beginning or has something changed along the way?

[aloubyansky]

Originally, runtime and deployment dependencies were in their own BOMs in Quarkus. But it's been awhile since they were merged together. For your extension we simply did it in the "universe" BOM. Going forward though, it'd be good to have it right in your original BOM.
Your BOM is also not required to import the quarkus-bom. The way your project is setup though currently, it may actually be useful to keep it for your internal build and tests.

Is the next release coming soon? Thanks!

[pivovarit]

Actually, I was supposed to release today so you were right on time with suggestions!

[aloubyansky]

Excellent! If you don't mind, once the release becomes available, please upgrade the version in the Quarkus platform repo and remove https://github.com/quarkusio/quarkus-platform/blob/main/bom/pom.xml#L133-L137. Thanks!