-
Notifications
You must be signed in to change notification settings - Fork 1
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
Improve Code documentation #71
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bitte sei noch so gut und vereinheitliche die Beispiele. Du verwendest 3 verschiende Quotes wild gemischt für die Viertel-Tags:
´<C_xxxx´
mit zwei Forward Ticks
"<AWF"
mit zwei double quotes
`<AHB´
mit back tick und forward tick.
Ich wäre für konsistent nur backticks.
Zugegeben war das bei den deutschen Begriffen vorher auch schon ein Mix aus single und double quotes. Aber wenn wir eh drüber gehen, dann doch am besten gleich einheitlich alles machen.
A "Segmentgruppe" inside an AHB, indicated by the ´<G_xxxx´ tag. | ||
This model can contain both Segments and segment groups. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ist das nicht sogar: G_SG?
zumindest für UTILTS und UTILMD ist das so:
return element.tag.startswith("G_SG") |
oder es klappt nur zufällig ;)
status_std: MigStatus # e.g. M | ||
status_specification: MigStatus # e.g. M |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hier hätte der mehrwert imho eher darin gelegen, das man die einzelne enum Members von MigStatus dokumentiert statt hier den ersten member als beispiel zu ergänzen.
No description provided.