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

Specification has no name #46

Closed
rubendel opened this issue Feb 9, 2022 · 4 comments
Closed

Specification has no name #46

rubendel opened this issue Feb 9, 2022 · 4 comments
Labels
agreed documentation Improvements or additions to documentation

Comments

@rubendel
Copy link
Contributor

rubendel commented Feb 9, 2022

<ids:specification use="optional" ifcVersion="IFC2X3"> <!-- this one will 'clash' with the previous one. We need some kind of 'alle values except' pattern. For example the ^ used in most regex languages, but not in XSD patterns -->

@berlotti
Copy link
Member

Good one!
Should this be mandatory?

@rubendel
Copy link
Contributor Author

I think it should be

@berlotti berlotti added agreed documentation Improvements or additions to documentation and removed to vote on labels Feb 15, 2022
@berlotti
Copy link
Member

Agreed in the meeting. Will process.

Documentation should motivate people to use unique names in an IDS file.

@atomczak
Copy link
Contributor

Agree on having the name mandatory.

I see a case where you might actually want to have duplicate names - when someone is not familiar with enumeration restriction, s/he might want to duplicate the clause and have results shown as if it was the same specification. There is always the possibility people don't use the tools as we would expect them to.

berlotti added a commit that referenced this issue Feb 15, 2022
#49 matching case
#46 made 'name' mandatory

- some general improvements
- updated examples
- made some fixes to xsd patterns
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
agreed documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

3 participants