Skip to content

Fix issue: unprintable character is rendered when handling comments in j2#149

Closed
stephenxs wants to merge 1 commit intomasterfrom
fix-unprintable
Closed

Fix issue: unprintable character is rendered when handling comments in j2#149
stephenxs wants to merge 1 commit intomasterfrom
fix-unprintable

Conversation

@stephenxs
Copy link
Copy Markdown
Owner

Use "{#-" and "-#}" to mark comments in jinja template

Why I did it

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

…n j2

Use "{#-" and "-#}" to mark comments in jinja template

Signed-off-by: Stephen Sun <stephens@nvidia.com>
@stephenxs
Copy link
Copy Markdown
Owner Author

ci 2924 + rerun 395 passed except SPC4 which is not supposed to run on ci so far.

@stephenxs stephenxs closed this Aug 28, 2023
@stephenxs stephenxs deleted the fix-unprintable branch August 28, 2023 14:43
stephenxs pushed a commit that referenced this pull request Sep 10, 2023
…lly (sonic-net#16480)

#### Why I did it
src/sonic-gnmi
```
* 6fd461c - (HEAD -> master, origin/master, origin/HEAD) Get origin from prefix (#149) (17 hours ago) [ganglv]
```
#### How I did it
#### How to verify it
#### Description for the changelog
stephenxs pushed a commit that referenced this pull request Mar 25, 2026
…ically (sonic-net#26286)

#### Why I did it
src/sonic-dash-ha
```
* 90c10b6 - (HEAD -> master, origin/master, origin/HEAD) Fix endianess mismatch. (#149) (10 hours ago) [dypet]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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