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
docs: use lowercase typings, use |
instead of or
in type declaration
#2167
Conversation
β Live Preview ready!
|
β Deploy Preview for nuxt-content ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
β Live Preview ready!
|
I've also seen that sometimes the types begin with a capital letter and sometimes they don't. E.g. here content/docs/content/4.api/3.configuration.md Lines 404 to 412 in 2ee6de5
See Boolean vs boolean .This also happens with other default types. I think they should all use lowercase letters, as that's the way they're used in code. The types I talk about are What do you think? |
Ok for me to have all lowercase, happy to update it on this PR? |
Sure. Will do that |
|
instead of or
in type declaration|
instead of or
in type declaration
I updated the typings for Content v2 now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks π
It would be great if you do so π |
Thanks @nobkd β€οΈ |
β Type of change
Other declarations follow the same schema.