Skip to content

Conversation

@pasched
Copy link
Contributor

@pasched pasched commented Sep 10, 2021

No description provided.

Signed-off-by: Patrick Schedlbauer <patrick.schedlbauer@parson-europe.com>
@pasched pasched requested review from a team, kmeids, pmai and stefancyliax September 10, 2021 12:21
* Added rules to the comments to enforce osi message validation with the https://github.com/OpenSimulationInterface/osi-validation[OSI-Validator].
* Added the rule definitions in the rules.yml.
* Modularized testing of commenting style into python unit tests.
* Defined a standardized file format for reading trace files (*.osi).
Copy link
Contributor

Choose a reason for hiding this comment

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

@pasched So now we fix typos in the release notes? 😃

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@stefancyliax In the heat of the review battle I just fixed them all and did not think about the agreement to leave them as they were. Please ignore these changes. ;)

Signed-off-by: Patrick Schedlbauer <patrick.schedlbauer@parson-europe.com>
Signed-off-by: Stefan Cyliax <stefan.cyliax@asam.net>
Copy link
Contributor

@stefancyliax stefancyliax left a comment

Choose a reason for hiding this comment

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

Looks fine now. Going to merge.

@stefancyliax stefancyliax merged commit d36ce88 into master Sep 10, 2021
@stefancyliax stefancyliax deleted the documentation/native-speaker-review branch September 10, 2021 18:54
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.

3 participants