-
Notifications
You must be signed in to change notification settings - Fork 50
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
Improve Documentation with Nullable #1095
Comments
Can you give a specific example of an API and fields you are talking about? In general it seems basically none of the API docs go to the detail of saying which fields are always there, sometimes there and such. |
For example, artifact config object's Sometimes the field didn't exist at all and was parsed as |
I must confess that it's such an immense undertaking given the # of APIs and having to go back to first principles to figure it out - that I wouldn't know where to start (but agree with its usefulness and that it's a good practice that'd be required if, say, publishing a more technical API specification like an OpenAPI spec). If there are some you have seen where it is documented, we could perhaps use that as a pattern to follow and start doing pieces incrementally. |
Ideally, I believe that documentation should be generated automatically from code to reduce operation costs and be more accurate. (Using OpenAPI also helps auto-generate the client code.) However, it will be difficult to migrate because api.go.cd also has agent documentation and has a large amount of APIs. If we take a light approach, after deciding how to display nullable properties as above coment, I think me and other contributors can fill in the missing details. |
Objects represented via
describe_object
may have fields that are nullable.Depending on the programming language (Python has
None
, TypeScript hasnull
, Go hasnil
, etc) there are cases where this nullability is important, so it would be nice if nullable fields are well documented.The text was updated successfully, but these errors were encountered: