Skip to content

Conversation

dwilkolek
Copy link
Contributor

Add a new configuration option typeSuffix and typePrefix to the CodeGenConfig class and update the code generation methods to append the affixes to the generated type names.

  • CodeGenConfig: Add a new configuration option typeSuffix and typePrefix to the CodeGenConfig class.
  • Java DataTypeGenerator: Update the generate method to append the typeSuffix and typePrefix to the generated type names.
  • Kotlin DataTypeGenerator: Update the generate method to append the typeSuffix and typePrefix to the generated type names.

based on: #773

@jstastny
Copy link

@srinivasankavitha -- could any of the reviewers take a look at this effort? We would also benefit from having the possiblity prefix / postfix the generated classes.
I am happy to try to resolve the conflicts to make this ready to merge again, but would like to know if there is any indication of somebody looking into this to avoid what happened to this PR, when @dwilkolek rebased the original #773.
Thanks a lot!

@dwilkolek dwilkolek force-pushed the amondnet/add-type-suffix branch from bd29e4b to 2d446d6 Compare August 13, 2025 21:01
@dwilkolek
Copy link
Contributor Author

dwilkolek commented Aug 13, 2025

I've rebased this PR onto last passing build.
Edit: oh, checks are run on pull-request. I'll wait for fix on master to rebase again 😉

…CodeGenConfig` class and update the code generation methods to append the affixes to the generated type names.

* **CodeGenConfig**: Add a new configuration option `typeSuffix` and `typePrefix` to the `CodeGenConfig` class.
* **Java DataTypeGenerator**: Update the `generate` method to append the `typeSuffix` and `typePrefix` to the generated type names.
* **Kotlin DataTypeGenerator**: Update the `generate` method to append the `typeSuffix` and `typePrefix` to the generated type names.
@dwilkolek dwilkolek force-pushed the amondnet/add-type-suffix branch from 2d446d6 to 6c81ef8 Compare August 25, 2025 21:01
@jstastny
Copy link

jstastny commented Sep 1, 2025

@dwilkolek -- thanks for the rebase effort.
@srinivasankavitha @paulbakker -- could somebody from the maintainers take a look at this one now that all checks are green? Thanks a lot.

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