Skip to content

refactor(build-logic): improve build configuration#2065

Merged
miurahr merged 4 commits into
masterfrom
topic/miurahr/build-system/improve-build-logic-abstract
Jun 3, 2026
Merged

refactor(build-logic): improve build configuration#2065
miurahr merged 4 commits into
masterfrom
topic/miurahr/build-system/improve-build-logic-abstract

Conversation

@miurahr
Copy link
Copy Markdown
Member

@miurahr miurahr commented Jun 3, 2026

We have the build-logic extended build configuration with groovy language and gradle convention mechanism.

There are several refactoring and improvement points in the build system.

Pull request type

  • Build and release changes -> [build/release]

Which ticket is resolved?

none

What does this PR change?

  • AbstractDocumentTask.groovy should be abstract and @Inject abstract ObjectFactory getObjects() to avoid referencing project in the logging configuration that is used in execution phase.
  • The inherited classes also become abstract
  • TransformationTask class uses deprecated class Resolver. This replace it with newer XMLResolver
  • We should not use other group in the configuration; whenb no group is specified, it goes to other
  • We don't need to create output directory when we specify outputs parameter; it is automatically created
  • Improve spotbugsReport task to deefine paths out of the execution task to support configuration cache.

Other information

miurahr added 4 commits June 3, 2026 11:56
…ve logging configuration

- Update `DocbookHtmlTask`, `TransformationTask`, and `WhcTask` to be abstract
- Refactor logging configuration in `AbstractDocumentTask` with better object usage to avoid referencing project object in execution phase
…subproject handling

- Avoid project referencing in execution block
…r` and update resource resolver initialization
@miurahr miurahr merged commit 7de870a into master Jun 3, 2026
14 checks passed
@miurahr miurahr deleted the topic/miurahr/build-system/improve-build-logic-abstract branch June 3, 2026 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant