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

Support for "date" type #3

Closed
pvgoran opened this issue Oct 28, 2018 · 3 comments
Closed

Support for "date" type #3

pvgoran opened this issue Oct 28, 2018 · 3 comments

Comments

@pvgoran
Copy link
Contributor

pvgoran commented Oct 28, 2018

OpenAPI supports the date format for the string type, but tinyspec doesn't have a corresponding type. It would be great to have it. (I would even go as far as redefining the d shorthand type to the new date type, and defining the new dt shorthand for the existing datetime type.)

P. S. I'm very new to the world of REST API specifications, but tinyspec already looks like a useful "source" language for OpenAPI, an attractive alternative to just writing these verbose specifications by hand.

@Ajaxy
Copy link
Owner

Ajaxy commented Oct 29, 2018

@pvgoran Thanks for the report. I think we can add the date type, without a shorthand, to not break the current API.

@pvgoran
Copy link
Contributor Author

pvgoran commented Oct 29, 2018

For some reason (lack of releases on github? very few issues created?), I assumed that the project might be too young to worry about breaking existing usage. However, now I see that the history goes back to year 2017, so such breaking changes shouldn't be done.

This was referenced Oct 29, 2018
@Ajaxy Ajaxy closed this as completed in #9 Nov 4, 2018
@Ajaxy
Copy link
Owner

Ajaxy commented Nov 4, 2018

Merged, thank you, @pvgoran!

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

No branches or pull requests

2 participants