Skip to content

refactor(template): simplify escaping go templates#23

Merged
JonZeolla merged 1 commit intomainfrom
normalize-jinja2-escaping
Jul 8, 2025
Merged

refactor(template): simplify escaping go templates#23
JonZeolla merged 1 commit intomainfrom
normalize-jinja2-escaping

Conversation

@JonZeolla
Copy link
Copy Markdown
Member

@JonZeolla JonZeolla commented Jul 8, 2025

Contributor Comments

This is just a refactor that simplifies how we escape the go templates inside of the template dir ({{...) to improve readability.

I tested a bunch of commands in a generated project and they all appeared to work as expected

task build
PLATFORM=all task build
task test
task sbom vulnscan license-check
task init
task lint
task validate
task update
$ ls sbom* vuln* license* zenable*
license-check.latest.linux_amd64.json                      sbom.latest.linux_arm64.spdx.json
license-check.latest.linux_arm64.json                      sbom.latest.linux_arm64.syft.json
sbom.latest.linux_amd64.cyclonedx.json                     vulns.latest.linux_amd64.json
sbom.latest.linux_amd64.spdx.json                          vulns.latest.linux_arm64.json
sbom.latest.linux_amd64.syft.json                          zenable-io_ai_native_python_testing_latest_all.tar
sbom.latest.linux_arm64.cyclonedx.json                     zenable-io_ai_native_python_testing_latest_linux_arm64.tar

Pull Request Checklist

Thank you for submitting a contribution!

Please address the following items:

  • If you are adding a dependency, please explain how it was chosen.
  • If manual testing is needed in order to validate the changes, provide a testing plan and the expected results.
  • Validate that documentation is accurate and aligned to any project updates or additions.

Copy link
Copy Markdown

@ai-coding-guardrails ai-coding-guardrails Bot left a comment

Choose a reason for hiding this comment

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

Nice work! 😎

I didn't find anything of concern

Reviewed with 🤟 by Zenable

@JonZeolla JonZeolla merged commit 4ae2af3 into main Jul 8, 2025
8 checks passed
@JonZeolla JonZeolla deleted the normalize-jinja2-escaping branch July 8, 2025 18:00
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.

1 participant