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

XEP-0388: Fix the XML Schema and examples #1369

Merged
merged 1 commit into from
Aug 23, 2024
Merged

Conversation

linkmauve
Copy link
Member

No description provided.

@linkmauve linkmauve changed the title XEP-0388: Add the XML Schema and fix examples XEP-0388: Fix the XML Schema and fix examples Aug 6, 2024
@github-actions github-actions bot added the Needs Council The affected XEP has the Council as Approving Body and it needs to decide on the change. label Aug 6, 2024
@linkmauve linkmauve changed the title XEP-0388: Fix the XML Schema and fix examples XEP-0388: Fix the XML Schema and examples Aug 6, 2024

<xs:simpleType name="Uuid">
<xs:restriction base="xs:string">
<xs:pattern value="([0-9]|[a-f]|[A-F]){8}-([0-9]|[a-f]|[A-F]){4}-([0-9]|[a-f]|[A-F]){4}-([0-9]|[a-f]|[A-F]){4}-([0-9]|[a-f]|[A-F]){12}"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a historical reason that ([0-9]|[a-f]|[A-F]){8} isn't written as ([0-9a-fA-F]){8} ?
Maybe compatibility with certain tools?

Copy link
Member Author

Choose a reason for hiding this comment

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

I probably just copy/pasted it from somewhere else, feel free to PR that change as it makes it slightly more readable. :)

@iNPUTmice iNPUTmice merged commit 90c887a into xsf:master Aug 23, 2024
1 check passed
@linkmauve linkmauve deleted the xep-0388 branch September 2, 2024 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Council The affected XEP has the Council as Approving Body and it needs to decide on the change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants