Skip to content

Unit present verbatim#4777

Merged
DagBruck merged 4 commits intomasterfrom
DagBruck-patch-1
Apr 24, 2026
Merged

Unit present verbatim#4777
DagBruck merged 4 commits intomasterfrom
DagBruck-patch-1

Conversation

@DagBruck
Copy link
Copy Markdown
Contributor

Fixes 3630.

@DagBruck DagBruck self-assigned this Apr 23, 2026
@DagBruck DagBruck added the bug Critical/severe issue label Apr 23, 2026
@DagBruck DagBruck linked an issue Apr 23, 2026 that may be closed by this pull request
@DagBruck DagBruck changed the title Dag bruck patch 1 Unit present verbatim Apr 23, 2026
Copy link
Copy Markdown
Contributor

@maltelenz maltelenz left a comment

Choose a reason for hiding this comment

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

Suggesting to use the more modern form with %{theThing}, but fine as is as well.

Comment thread Modelica/Blocks/Interfaces.mo Outdated
Comment thread Modelica/Blocks/MathBoolean.mo Outdated
Copy link
Copy Markdown
Contributor

@HansOlsson HansOlsson left a comment

Choose a reason for hiding this comment

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

Looks good to me.

I tried to search for textString=.*%[a-zA-Z]* and didn't find any similar ones, so I believe this suffices.

DagBruck and others added 2 commits April 23, 2026 21:01
Co-authored-by: Malte Lenz <maltel@wolfram.com>
Co-authored-by: Malte Lenz <maltel@wolfram.com>
@DagBruck
Copy link
Copy Markdown
Contributor Author

@maltelenz @HansOlsson I accepted the proposed change, but unfortunately your approvals are now stale. Please re-approve.

@DagBruck DagBruck merged commit 5a2f6d8 into master Apr 24, 2026
20 checks passed
@DagBruck DagBruck deleted the DagBruck-patch-1 branch April 24, 2026 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Critical/severe issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unit present verbatim in text macro

3 participants