-
Notifications
You must be signed in to change notification settings - Fork 0
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
<para> Note appears 4 times #16
Comments
@jh-isw thanks for having a look! I thought that was the intended way to mark the different paragraphs? |
I meant it's 4 times more or less the same content:
|
Thanks! This is an issue with indention in the docstring of aas-core-meta: https://github.com/aas-core-works/aas-core-meta/blob/2a81612dab519215621c55c8e3b434f441e5f588/aas_core_meta/v3.py#L4375. The notes should be indented one level more. I'll try to fix this tomorrow. |
We erroneously intertwined the remarks and constraints in some of the docstrings. Moreover, some notes were not properly indented, and were thus parsed as remarks instead of notes referring to the respective constraint. This patch fixes the issue. This is directly related to [the issue #16 in aas-core3.0-csharp]. [the issue #16 in aas-core3.0-csharp]: aas-core-works/aas-core3.0-csharp#16
We erroneously intertwined the remarks and constraints in some of the docstrings. Moreover, some notes were not properly indented, and were thus parsed as remarks instead of notes referring to the respective constraint. This patch fixes the issue. This is directly related to [the issue #16 in aas-core3.0-csharp]. [the issue #16 in aas-core3.0-csharp]: aas-core-works/aas-core3.0-csharp#16
see this line
aas-core3.0-csharp/src/AasCore.Aas3_0/types.cs
Line 10173 in 99870fd
and the following
The text was updated successfully, but these errors were encountered: