Skip to content

[patch] Normalize known_issues entry formatting - #180

Closed
prveenkumr wants to merge 1 commit into
stablefrom
known-issue-fix
Closed

[patch] Normalize known_issues entry formatting#180
prveenkumr wants to merge 1 commit into
stablefrom
known-issue-fix

Conversation

@prveenkumr

Copy link
Copy Markdown
Contributor

No description provided.

@prveenkumr
prveenkumr requested a review from a team as a code owner February 2, 2026 09:36

@whitfiea whitfiea left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We shouldn't change the contents as it will just break again when we add another line with & set. Instead we should replace & with &amp here https://github.com/ibm-mas/cli/blob/master/python/src/mas/cli/displayMixins.py#L47 (note that content is a list so we need to iterate around it) just like we do for other print statements https://github.com/ibm-mas/cli/blob/master/python/src/mas/cli/displayMixins.py#L53

@IanBoden
IanBoden self-requested a review February 2, 2026 10:08
@durera

durera commented Feb 2, 2026

Copy link
Copy Markdown
Contributor

Fix is here: ibm-mas/cli#2031

@durera durera closed this Feb 2, 2026
@durera
durera deleted the known-issue-fix branch February 2, 2026 11:49
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.

4 participants