Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ability to supply a root directory with custom templates: "customTemplatesRoot" #1198

Merged
merged 7 commits into from
May 2, 2023
Merged

Ability to supply a root directory with custom templates: "customTemplatesRoot" #1198

merged 7 commits into from
May 2, 2023

Conversation

isaac-mercieca
Copy link
Contributor


Description

The custom template option doesn't work correctly with the Gradle Plugin since the daemon folder is the root instead of the project folder.

The solution included in this pull request includes support for a new configuration customTemplatesRoot. This will allow the developer to configure the template directory for a project. Since the configuration is available from each of the build plugins the default value for the template's root will be defined by each plugin as the project's root directory.

Related to #1158


Changes were made to:

  • Codegen library - Java
  • Codegen library - Kotlin
  • Codegen library - Scala
  • Maven plugin
  • Gradle plugin
  • SBT plugin

@kobylynskyi kobylynskyi added this to the 5.8.0 milestone May 2, 2023
@kobylynskyi
Copy link
Owner

@isaac-mercieca looks good! Thanks for the contribution.

@kobylynskyi kobylynskyi changed the title Gradle Custom Templates Ability to supple a root directory with custom templates May 2, 2023
@kobylynskyi kobylynskyi changed the title Ability to supple a root directory with custom templates Ability to supply a root directory with custom templates: "customTemplatesRoot" May 2, 2023
@kobylynskyi kobylynskyi merged commit d3eb4c9 into kobylynskyi:main May 2, 2023
@isaac-mercieca isaac-mercieca deleted the gradle-custom-templates branch May 2, 2023 20:12
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.

2 participants