Skip to content
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

[ENHANCEMENT] add kubebuilder annotations to dashboard types #1764

Merged

Conversation

jgbernalp
Copy link
Contributor

Description

This PR adds kubebuilder annotations to dashboard related types, this allows that the Dashboard CRD for Perses is generated correctly

Checklist

  • Pull request has a descriptive title and context useful to a reviewer.
  • Pull request title follows the [<catalog_entry>] <commit message> naming convention using one of the
    following catalog_entry values: FEATURE, ENHANCEMENT, BUGFIX, BREAKINGCHANGE, DOC,IGNORE.
  • All commits have DCO signoffs.

UI Changes

no UI Changes

Extension FileExtension `json:"extension" yaml:"extension"`
CaseSensitive bool `json:"case_sensitive" yaml:"case_sensitive"`
Folder string `json:"folder" yaml:"folder"`
Extension FileExtension `json:"extension" yaml:"extension"`
Copy link
Member

Choose a reason for hiding this comment

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

Extension is also optional if you want to annotate it

@Nexucis
Copy link
Member

Nexucis commented Feb 16, 2024

regarding the issue with the doc, I have tackled in a different PR: #1765

Signed-off-by: Gabriel Bernal <gbernal@redhat.com>
@jgbernalp jgbernalp force-pushed the add-kubebuilder-annotations-for-dashboards-crds branch from a205028 to 5ef9950 Compare February 19, 2024 08:28
@Nexucis Nexucis added this pull request to the merge queue Feb 19, 2024
Merged via the queue into perses:main with commit 6799f0a Feb 19, 2024
18 checks passed
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.

2 participants