Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unnecessary whitespace for selective model extension #46

Closed
m-kessler opened this issue Jun 4, 2024 · 1 comment · Fixed by #47
Closed

Unnecessary whitespace for selective model extension #46

m-kessler opened this issue Jun 4, 2024 · 1 comment · Fixed by #47
Assignees
Labels
bug Something isn't working
Milestone

Comments

@m-kessler
Copy link
Contributor

The selective model extension, which mofmt supports since #6, is not formatted correct.

This code:

model NewPlant
  extends System(
    break controller,
    break reference);
end NewPlant;

Is formatted as:

model NewPlant

  extends System(
    break controller ,
    break reference );

end NewPlant;

Note the extra white spaces at the end of each break statement.

@ErykMroczek ErykMroczek added the bug Something isn't working label Jun 4, 2024
@ErykMroczek ErykMroczek self-assigned this Jun 4, 2024
ErykMroczek added a commit that referenced this issue Jun 10, 2024
@ErykMroczek ErykMroczek added this to the 0.6.0 milestone Jun 10, 2024
@ErykMroczek
Copy link
Owner

Fixed in d3c7293, so you can use this commit in pre-commit. For manual usage, use command cargo install --git.

It will be included in 0.6.0, but I will wait with the release until I implement HTML formatting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants