Skip to content

Support non-ascii (unicode) enum and type names. #2038

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

Closed
wants to merge 47 commits into from

Conversation

josstn
Copy link

@josstn josstn commented Feb 14, 2024

This replaces regexp patterns that only worked with ascii characters with more proper matching that supports unicode identifiers in typescript/javascript.

The platform must support "unicode-aware mode" (the u flag) for this to work. https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp/unicode

nicolas-chaulet and others added 30 commits January 10, 2024 11:28

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Adding const support

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Optional response type when 204 response exists

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…type

test(snapshot): fix snapshot

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…vices-regexp

feat(client): support regexp to select services to export

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…ction

Adding a unittest action

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
test(ci): match steps from CircleCI

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…regexp

fix(client): support regexp to select models to export

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Introduce a couple of broken schema parsing

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
mrlubos and others added 15 commits February 3, 2024 07:03

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…-option

fix(config): support autoformat option flag

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
fix(parser): parse array items only if parent has no other item type

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…-properties

fix(parser): type additional properties with properties

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
fix(api): allow overriding request body name with x-body-name key

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was signed with the committer’s verified signature.
josstn Jostein Stuhaug
This replaces regexp patterns that only worked with ascii characters with more proper matching that supports unicode identifiers in typescript/javascript.

The platform must support "unicode-aware mode" (the u flag) for this to work. https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RegExp/unicode

Verified

This commit was signed with the committer’s verified signature.
josstn Jostein Stuhaug
So that it matches the desired test result.
@mrlubos
Copy link
Collaborator

mrlubos commented Feb 14, 2024

@josstn this looks great! Want to open a pull request in our fork too? Otherwise I might merge this myself at some point, great work!

@josstn
Copy link
Author

josstn commented Feb 14, 2024

@josstn this looks great! Want to open a pull request in our fork too? Otherwise I might merge this myself at some point, great work!

Sure: hey-api/openapi-ts#14

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@josstn
Copy link
Author

josstn commented Feb 16, 2024

Replaced by #2040

@josstn josstn closed this Feb 16, 2024
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

3 participants