Add Helm tech doc analysis

[paigecalvert]

No description provided.

[netlify]

✅ Deploy Preview for cncf-techdocs ready!

Name	Link
🔨 Latest commit	1bb91cc
🔍 Latest deploy log	https://app.netlify.com/projects/cncf-techdocs/deploys/695fe6141350750008bee990
😎 Deploy Preview	https://deploy-preview-332--cncf-techdocs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[nate-double-u]

This is great @paigecalvert, thanks!
Let us know when you'd like help reviewing it 🙂

[paigecalvert]

Thank you Nate will do! Just wanted to review a couple things with one of the Helm maintainers. Will ping you all soon :)

This is great @paigecalvert, thanks!
Let us know when you'd like help reviewing it 🙂

[thisisobate]

Thanks @paigecalvert for taking out time to do this analysis. This is very well detailed and fulfills our analysis criteria.

One thing I noticed in the proposed sidebar noted in the implementaion file is the absence of the API/CLI reference. I think we should make it easier for developers to access the reference docs regardless of where it lives.

Other than that, I think this is good to go!

[thisisobate]

@paigecalvert I noticed you didn't add the helm-issues-list.md to this PR. Would you want to add it back?

[paigecalvert]

@paigecalvert I noticed you didn't add the helm-issues-list.md to this PR. Would you want to add it back?

Yeah can do. I was going to work on creating all of the issues as part of a follow-up iteration after checking back in with the helm team, but I can certainly get some placeholders in there now if that sounds good to you?

[thisisobate]

Sounds perfect!

[paigecalvert]

^ left a couple TO DOs for adding in the links once those issues are in github

[thisisobate]

Looks like there are some formatting and spelling issues causing CI to break. Can you try and fix those too so we can merge? @paigecalvert