-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Replace gen-crd-api-reference-docs
for API documentation generation
#6144
Comments
Fixes prometheus-operator#6144 Signed-off-by: Jayapriya Pai <slashpai9@gmail.com>
I cleared my assignment as I didnt get enough time to work on this one but this can be a good one to get started for GSoc :) |
@slashpai maybe a good first issue tag so that folks can find? |
I haven't put the label before because I like CNCF's guidelines for labeling something with good-first-issue, I'll try to improve the issue description later 😬 |
@ArthurSens sure, I think since replacing the gen library doesn't need to have prometheus-operator knowledge much it can be a tackle for someone familiar with go as well. Do you want to remove good first label on this? |
nah let's keep it :) |
just showing interest for working on this (already started in fact). I will follow up with a PR in the upcoming days. |
I have been experimenting with elastic/crd-ref-docs today and here are some important notes I took:
@slashpai @ArthurSens your thoughts on this? |
Thanks for taking a look!
Yeah, that was expected I guess... no problems in changing everything, it's just a lot of work 😬
Oh damn, that's a boomer. I'm not sure what would be the impact if we change this, maybe that's fine to change?
That's an easy fix :) no problems adding that annotation and no problems sorting things alphabetically |
#6400 has been fixed. Seems like that has fixed this issue too? |
Yes definitely. I thought the two issues were closed. However, if embedded struct pointers are used again the issue may reappear. That's why it is still open I guess. |
The tool we're using is still deprecated, in my opinion we still want to replace it 😬 |
In #6001 we noticed that inline structs in our API are not reflected by gen-crd-api-reference-docs.
The author of gen-crd-api-reference-docs also mentions how the tool is not maintained anymore and other options, like elastic/crd-ref-docs, have already fixed the inline structs problem.
We want to move away from gen-crd-api-reference-docs for the reasons above
The text was updated successfully, but these errors were encountered: