Skip to content

fix(generator): align template and post-processor formatting#13028

Open
blakeli0 wants to merge 1 commit intomainfrom
fix/ci-owlbot
Open

fix(generator): align template and post-processor formatting#13028
blakeli0 wants to merge 1 commit intomainfrom
fix/ci-owlbot

Conversation

@blakeli0
Copy link
Copy Markdown
Contributor

@blakeli0 blakeli0 commented May 6, 2026

This PR aligns the owlbot.py.j2 template with the formatting expected by update_owlbot_postprocessor_config.sh (no trailing comma on the last element and ]) closing tag), and synchronizes the existing owlbot.py files.

@blakeli0 blakeli0 requested a review from a team as a code owner May 6, 2026 19:23
@blakeli0 blakeli0 changed the title fix(owlbot): align template and post-processor formatting fix(generator): align template and post-processor formatting May 6, 2026
@blakeli0 blakeli0 requested a review from jinseopkim0 May 6, 2026 19:25
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request modifies the formatting of owlbot.py and its Jinja2 template, specifically adjusting list item commas and closing brackets. A high-severity syntax error was identified in the template logic that would result in invalid Python code when template_excludes is provided, as it generates redundant closing parentheses. A fix was suggested to correctly handle the conditional closing of the function call.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 6, 2026

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 6, 2026

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