Support rendering consts that are enum values for a type defined in package by liorokman · Pull Request #79 · elastic/crd-ref-docs · GitHub
Skip to content

Support rendering consts that are enum values for a type defined in package#79

Merged
thbkrkr merged 2 commits into
elastic:masterfrom
liorokman:const-values
Apr 13, 2024
Merged

Support rendering consts that are enum values for a type defined in package#79
thbkrkr merged 2 commits into
elastic:masterfrom
liorokman:const-values

Conversation

@liorokman

Copy link
Copy Markdown
Contributor

Add support to extract information about const values of aliased enumeration types.

Fixes #54

@cla-checker-service

cla-checker-service Bot commented Apr 10, 2024

Copy link
Copy Markdown

@liorokman

Copy link
Copy Markdown
Contributor Author

/retest

@liorokman

Copy link
Copy Markdown
Contributor Author

@thbkrkr thbkrkr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good overall, maybe take care of the naming a little more and see how to improve the readability of this new function.

Comment thread processor/processor.go Outdated
Comment thread types/types.go Outdated
Comment thread processor/processor.go Outdated

@thbkrkr thbkrkr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Thank you @liorokman.

@thbkrkr thbkrkr merged commit ea9fcaa into elastic:master Apr 13, 2024
@liorokman liorokman deleted the const-values branch April 14, 2024 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support rendering consts that are enum values for a type defined in package

2 participants