-
Notifications
You must be signed in to change notification settings - Fork 68
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
Default id
to "Read-Only" + allow for empty blocks/nested attributes to be documented
#134
Conversation
This still allows arbitrary definition of `id` where a tailor-made `.Description` can be provided: it's just the default that is changing to something more sensible. Also, updated README to guide developers. Co-authored-by: Nicholas Muesch <nicholas.muesch@datadoghq.com> Co-authored-by: Oliver Ford <dev.github@ojford.com>
id
to "Read-Only" + allow for empty blocks/nested attributes to be documented
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.
LGTM
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.
Looks good to me 🚀
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
Closes #46
Closes #99
This PR is a cherry-pick-like one: there are key fixes we want to land in
v0.8.0
, and given PRs have been open for a long while, we want to support them without additional burden to the original contributors.Of course, authorship is preserved.
See the linked content for more context about each fix.
Big thank you to: @ovandriyanov, @nmuesch, @OJFord