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

Edited schema with no functional change. #170

Closed

Conversation

CBenghi
Copy link
Contributor

@CBenghi CBenghi commented Jun 19, 2023

This was done to remove xs:extensions where they were not used.

This PR relates to #162

This was done to remove `xs:extensions` where they were not used.
<xs:complexType>
<xs:complexContent>
<xs:extension base="ids:requirementsType">
<xs:attribute name="description" type="xs:string" use="optional"/>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The requirementsType was referenced only here, so no need to add the description via extension, I've added it directly to the type specification.

@berlotti
Copy link
Member

I think this was about putting 'instructions' on the 'requirements' level so it would be inherited by 'entity', 'partof', etc... under requirements.
I see this pull request removing the 'instructions' from the individual parts, but not adding it to 'requirements' in a way it is being inherited.

@CBenghi
Copy link
Contributor Author

CBenghi commented Jun 20, 2023

@berlotti
sorry, I'm not getting your point.
I don't see any changes with regards to instructions in any line change of these PR.
Can you elaborate?

@berlotti
Copy link
Member

sorry; mixing up 'instructions' and 'description'.
all good now.

@berlotti
Copy link
Member

fixed in master (this pull request was out of sync)

@berlotti berlotti closed this Jun 20, 2023
@CBenghi CBenghi deleted the simplifySchemaExtensions branch June 22, 2023 06:43
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.

None yet

2 participants