LGTM. We need to be careful with the timing of merging this PR, if we merge it now and someone regenerates showcase with the existing setup, it would be overridden. Maybe try to include this change in your showcase PR?

We need to be careful with the timing of merging this PR, if we merge it now and someone regenerates showcase with the existing setup, it would be overridden.

The required showcase golden tests should fail until then. Please do not merge until ci / showcase (11) and (17) tests are passing.

Is it possible to modify the current showcase generation config to generate with comments?

Yeah I believe that's what Diego is working on next.

@blakeli0 @meltsufin I included the necessary command to include comments with the current showcase configuration.

@diegomarquezp : The --experimental... argument needs to also be added to the //showcase:verify_grpc configuration, directly above the //showcase:update_grpc profile you've already edited.

thanks @burkedavison. I added the option to such profiles

[java_showcase_unit_tests] Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication