refactor(tts): Remove tts_synthesize_text_audio_profile duplicated region tag#10311
Conversation
There was a problem hiding this comment.
Code Review
This pull request removes the region tags for tts_synthesize_text_audio_profile in SynthesizeText.java. The review feedback points out that removing the start tag also removed a blank line, violating the Google Java Style Guide's vertical whitespace recommendation between class members, and suggests restoring it.
|
Here is the summary of changes. You are about to delete 1 region tag.
This comment is generated by snippet-bot.
|
amcolin
left a comment
There was a problem hiding this comment.
As you mentioned this tag is duplicated, and the documentation is currently referencing the file.texttospeech/snippets/src/main/java/com/example/texttospeech/SynthesizeTextBeta.java so please go ahead.
Description
Fixes #
b/536065373
Removes duplicated region tag
tts_synthesize_text_audio_profilefromtexttospeech/snippets/src/main/java/com/example/texttospeech/SynthesizeText.javato prevent region tag collisions and ensure accurate snippet mapping in documentation.NOTE: Don't remove file because it is used and linked to other region tags, which if neccessary will be treated in their own bug.
Checklist
Testing
mvn clean verifyrequiredmvn -P lint checkstyle:checkrequiredmvn -P lint clean compile pmd:cpd-check spotbugs:checkadvisory onlyCompliance & Style
pom.xmlparent set to latestshared-configurationPost-Approval Actions