Insert Spaces in Modelica.Blocks.Sources.mo by MatthiasBSchaefer · Pull Request #2 · LTX-Shadow/ModelicaStandardLibrary

MatthiasBSchaefer · 2026-06-30T08:40:35Z

for better readability

Summary by CodeRabbit

Style
- Reformatted spacing for default output connector values in expression blocks (RealExpression, IntegerExpression, BooleanExpression) for consistency (e.g., y=0.0/y=0/y=false → y = ...); no functional behavior changed.
- Adjusted spacing in the BooleanExpression dialog group text for consistency.

for better readability

coderabbitai · 2026-06-30T08:40:55Z

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

@coderabbitai resume to resume automatic reviews.
@coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

▶️ Resume reviews
🔍 Trigger review

📝 Walkthrough

Walkthrough

Modelica/Blocks/Sources.mo only changes spacing in default-value and parameter declarations, plus one annotation argument. No values, equations, or behavior changed.

Changes

Formatting Fix

Layer / File(s)	Summary
Expression whitespace adjustment `Modelica/Blocks/Sources.mo`	Adds spaces around `=` in the `y` declarations for `RealExpression`, `IntegerExpression`, and `BooleanExpression`; `BooleanExpression` annotation spacing and `Constant` parameter initialization spacing are also reformatted.
Estimated code review effort: 1 (Trivial)	~1 minute

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately reflects the PR's main change: whitespace formatting updates in Modelica.Blocks.Sources.mo.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch Test_RegressionTestingCI

_{Comment @coderabbitai help to get the list of available commands.}

MSL-RegressionTesting-CI

Please look here for details:https:/serv.ltx.de/prs/2/Modelica/PR_comparison_report.html and https:/serv.ltx.de/prs/2/ModelicaTest/PR_comparison_report.html

MSL-RegressionTesting-CI

Please look here for details:https:/serv.ltx.de/prs/2/Modelica/PR_comparison_report.html and https:/serv.ltx.de/prs/2/ModelicaTest/PR_comparison_report.html

MSL-RegressionTesting-CI

Please look here for details:https:/serv.ltx.de/prs/2/Modelica/PR_comparison_report.html and https:/serv.ltx.de/prs/2/ModelicaTest/PR_comparison_report.html

MSL-RegressionTesting-CI

Please look here for details:https:/serv.ltx.de/prs/2/Modelica/PR_comparison_report.html and https:/serv.ltx.de/prs/2/ModelicaTest/PR_comparison_report.html

MSL-RegressionTesting-CI

Please look here for details:https://serv.ltx.de/prs/2/Modelica/PR_comparison_report.html and https://serv.ltx.de/prs/2/ModelicaTest/PR_comparison_report.html

MSL-RegressionTesting-CI

Please look here for details:https://serv.ltx.de/prs/2/Modelica/PR_comparison_report.html and https://serv.ltx.de/prs/2/ModelicaTest/PR_comparison_report.html

MSL-RegressionTesting-CI

Please look here for details:https://serv.ltx.de/prs/2/Modelica/PR_comparison_report.html and https://serv.ltx.de/prs/2/ModelicaTest/PR_comparison_report.html

MSL-RegressionTesting-CI

Please look here for details: https://serv.ltx.de/prs/2/Modelica/PR_comparison_report.html and https://serv.ltx.de/prs/2/ModelicaTest/PR_comparison_report.html

MSL-RegressionTesting-CI

Please look here for details: https://serv.ltx.de/prs/2/Modelica/PR_comparison_report.html and https://serv.ltx.de/prs/2/ModelicaTest/PR_comparison_report.html

MSL-RegressionTesting-CI

Please look here for details: https://serv.ltx.de/prs/2/Modelica/PR_comparison_report.html and https://serv.ltx.de/prs/2/ModelicaTest/PR_comparison_report.html

MSL-RegressionTesting-CI

Please look here for details:https://serv.ltx.de/prs//2/Modelica/PR_comparison_report.html and https://serv.ltx.de/prs//2/ModelicaTest/PR_comparison_report.html

MSL-RegressionTesting-CI

Please look here for details:https://serv.ltx.de/prs//2/Modelica/PR_comparison_report.html and https://serv.ltx.de/prs//2/ModelicaTest/PR_comparison_report.html

MSL-RegressionTesting-CI

Please look here for details:https://serv.ltx.de/prs//2/Modelica/PR_comparison_report.html and https://serv.ltx.de/prs//2/ModelicaTest/PR_comparison_report.html

MSL-RegressionTesting-CI

Please look here for details:https://serv.ltx.de/prs//2/Modelica/PR_comparison_report.html and https://serv.ltx.de/prs//2/ModelicaTest/PR_comparison_report.html

Insert Spaces in Modelica.Blocks.Sources.mo

62cf312

for better readability

MatthiasBSchaefer added 5 commits July 1, 2026 10:27

Fix formatting of RealOutput declaration in Sources.mo

0c6865e

Fix formatting of BooleanOutput declaration

7a209a1

Fix annotation formatting in BooleanExpression block

bf914ed

Fix formatting of parameter declaration in Constant block

46e1326

Fix annotation formatting in Sources.mo

e4186e4

MSL-RegressionTesting-CI suggested changes Jul 1, 2026

View reviewed changes

MSL-RegressionTesting-CI approved these changes Jul 1, 2026

View reviewed changes

MSL-RegressionTesting-CI suggested changes Jul 2, 2026

View reviewed changes

Fix graphics extent in Sources.mo annotation

d890fc7

MSL-RegressionTesting-CI suggested changes Jul 10, 2026

View reviewed changes

MSL-RegressionTesting-CI suggested changes Jul 13, 2026

View reviewed changes

Uh oh!

Conversation

MatthiasBSchaefer commented Jun 30, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

coderabbitai Bot commented Jun 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Reviews paused

Walkthrough

Changes

Uh oh!

MSL-RegressionTesting-CI left a comment

Choose a reason for hiding this comment

Uh oh!

MSL-RegressionTesting-CI left a comment

Choose a reason for hiding this comment

Uh oh!

MSL-RegressionTesting-CI left a comment

Choose a reason for hiding this comment

Uh oh!

MSL-RegressionTesting-CI left a comment

Choose a reason for hiding this comment

Uh oh!

MSL-RegressionTesting-CI left a comment

Choose a reason for hiding this comment

Uh oh!

MSL-RegressionTesting-CI left a comment

Choose a reason for hiding this comment

Uh oh!

MSL-RegressionTesting-CI left a comment

Choose a reason for hiding this comment

Uh oh!

MSL-RegressionTesting-CI left a comment

Choose a reason for hiding this comment

Uh oh!

MSL-RegressionTesting-CI left a comment

Choose a reason for hiding this comment

Uh oh!

MSL-RegressionTesting-CI left a comment

Choose a reason for hiding this comment

Uh oh!

MSL-RegressionTesting-CI left a comment

Choose a reason for hiding this comment

Uh oh!

MSL-RegressionTesting-CI left a comment

Choose a reason for hiding this comment

Uh oh!

MSL-RegressionTesting-CI left a comment

Choose a reason for hiding this comment

Uh oh!

MSL-RegressionTesting-CI left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

MatthiasBSchaefer commented Jun 30, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Jun 30, 2026 •

edited

Loading