Skip to content

Dedicated chapter about code generation configuration#1850

Merged
renejeglinsky merged 42 commits intomainfrom
codegen-switches-explained
Aug 26, 2025
Merged

Dedicated chapter about code generation configuration#1850
renejeglinsky merged 42 commits intomainfrom
codegen-switches-explained

Conversation

@vmikhailenko
Copy link
Contributor

@vmikhailenko vmikhailenko commented May 15, 2025

Introduces chapter to describe switches in Java Code generator. Previously that was documented in the Maven Plugin. This chapter somewhat repeats that documentation and aims to improve the visibility of said switches.

@vmikhailenko vmikhailenko marked this pull request as ready for review May 16, 2025 14:35
@vmikhailenko vmikhailenko requested a review from smahati as a code owner May 16, 2025 14:35
@vmikhailenko vmikhailenko requested a review from beckermarc May 19, 2025 08:59
@BraunMatthias
Copy link
Contributor

@beckermarc : is the review done and the PR ready for approval?

@beckermarc
Copy link
Contributor

@beckermarc : is the review done and the PR ready for approval?

I mentioned in the daily already multiple times that I don't have much time to look into this PR and asked others to jump in here. Nevertheless I read over it again now and left two comments :)

@vmikhailenko vmikhailenko requested a review from agoerler July 31, 2025 15:20
vmikhailenko and others added 6 commits August 22, 2025 12:34
Co-authored-by: René Jeglinsky <rene.jeglinsky@sap.com>
Co-authored-by: René Jeglinsky <rene.jeglinsky@sap.com>
Co-authored-by: René Jeglinsky <rene.jeglinsky@sap.com>
Co-authored-by: René Jeglinsky <rene.jeglinsky@sap.com>
# Conflicts:
#	java/developing-applications/building.md
@vmikhailenko
Copy link
Contributor Author

@renejeglinsky Can we merge this?

@renejeglinsky
Copy link
Contributor

Hi @smahati ,
please do a review later. Merging this now as it's already a huge PR. Thanks :)

@renejeglinsky renejeglinsky merged commit dce3c76 into main Aug 26, 2025
7 checks passed
@renejeglinsky renejeglinsky deleted the codegen-switches-explained branch August 26, 2025 11:15
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.

6 participants