handlebars.java java modules: modularize handlebars modules

Oct 18 '22 13:10 jknack

@jknack is there any chance you can pump out a release with at least Automatic-Module-Name ?

I need this kind of badly for our general conversion to Jooby 3.0

Jul 19 '23 17:07 agentgt

Or I can put in a PR if you like?

Jul 19 '23 17:07 agentgt

@jknack, any updates on this enhancement? I am currently unable to modularize my application because I use WireMock, which in turn uses handlebars and handlebars-helpers. This causes a compilation error: [ERROR] module wiremock reads package com.github.jknack.handlebars.helper from both handlebars.helpers and handlebars

It's important that I get this resolved, soon. I could lend a hand, if so desired, to modularize this code. I imagine renaming the above package would be a breaking change for many users, but I believe it will need to be done to complete this enhancement.

Oct 17 '23 21:10 Montimur

@jknack I put in a PR on the 4.x branch to try to make it as easy as possible.

Oct 17 '23 22:10 agentgt

hey guys, I don't have time to work on this yet... but I feel we should truly write modules not just adding the manifest entry and set JDK 17 as minimum requirement.

Oct 18 '23 15:10 jknack

I agree. I just wanted to help @Montimur and others that can't wait.

I can possibly do a module-info PR in master but maybe there is more work that you want to do prior (e.g. refactoring)?

Oct 18 '23 15:10 agentgt

right, requires a bit more (cleanup mainly). Next release should be "modern JVM friendly", nothing more.

Oct 18 '23 15:10 jknack