feat: add new functions to cel-go docs

flanksource / docs

https://docs.flanksource.com/

0 stars 1 forks source link

feat: add new functions to cel-go docs #109

Closed adityathebe closed 8 months ago

adityathebe commented 8 months ago

resolves: https://github.com/flanksource/docs/issues/106

netlify[bot] commented 8 months ago

Deploy Preview for canarychecker ready!

Name	Link
Latest commit	901c1d84ebe8ab8b99b31580c5e041468dcf5bc7
Latest deploy log	https://app.netlify.com/sites/canarychecker/deploys/657ec2ce485fa40007f50103
Deploy Preview	https://deploy-preview-109--canarychecker.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 site configuration.

netlify[bot] commented 8 months ago

Deploy Preview for flanksource-docs canceled.

Name	Link
Latest commit	901c1d84ebe8ab8b99b31580c5e041468dcf5bc7
Latest deploy log	https://app.netlify.com/sites/flanksource-docs/deploys/657ec2ce68601b0008b0c26d

adityathebe commented 8 months ago

@moshloop Should we remove functions that are already methods?

Eg: toJSON, JSON, JSONArray.

I think we could also remove humanDuration/humanSize since we already have HumanDuration/HumanSize. From what I am seeing, we can go with the convention of

methods -> camel case
functions -> Pascal case

Also we could namespace the aws and k8s functions. k8s.GetStatus instead of GetStatus

github-actions[bot] commented 8 months ago

:tada: This PR is included in version 1.0.36 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket:

flanksource / docs