Skip to content

Move annotation processor to its own module - #55

Merged
jdcasey merged 1 commit into
Commonjava:masterfrom
dwalluck:groovy-templates
Aug 24, 2026
Merged

Move annotation processor to its own module#55
jdcasey merged 1 commit into
Commonjava:masterfrom
dwalluck:groovy-templates

Conversation

@dwalluck

Copy link
Copy Markdown
Contributor

Upgrade groovy-templates and fix scope.

Make generated output have deterministic order.

@dwalluck

Copy link
Copy Markdown
Contributor Author

The processor was mixed into the core rwx module and pulled in groovy-templates as a dependency even though it was not required.

@rnc

rnc commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

@jdcasey https://github.com/release-engineering/kojiji/ uses RWX and I think Indy does as well. This change simplifies the dependency tree to avoid bringing in older dependencies (e.g. groovy). I don't know how Indy uses it. I am ok with these changes.

@rnc
rnc requested a review from jdcasey August 20, 2026 07:24
@dwalluck

Copy link
Copy Markdown
Contributor Author

This one is just a refactor to move the anno proc to build-time. The pom version change was in #56.

Upgrade groovy-templates and fix scope.

Make generated output have deterministic order.
@jdcasey
jdcasey merged commit b8ff142 into Commonjava:master Aug 24, 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