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

Fix String type with byte format #25

Merged
merged 1 commit into from Feb 17, 2022
Merged

Fix String type with byte format #25

merged 1 commit into from Feb 17, 2022

Conversation

mattia
Copy link
Contributor

@mattia mattia commented Feb 17, 2022

When the spec defines a string property with a byte format at the moment a String property is generated, but a Data property should be generated instead.

With input:

test_property:
  type: string
  format: byte

Before:

var testProperty: String

After:

var testProperty: Data

@kean kean merged commit c6aad2c into CreateAPI:main Feb 17, 2022
@kean
Copy link
Member

kean commented Feb 17, 2022

Thanks for making the change! I appreciate it.

@mattia
Copy link
Contributor Author

mattia commented Feb 17, 2022

Thank you for the library!

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