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

pulling projectsveltos CRDs of version defined in Chart.yaml #1041

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

BROngineer
Copy link
Contributor

Fixes: #1040

@BROngineer BROngineer force-pushed the chore/issue-1040-maintain-sveltos-crds-version branch 2 times, most recently from 9fb9f57 to 4febff2 Compare February 5, 2025 05:08
Makefile Outdated Show resolved Hide resolved
Makefile Outdated
generate-all: generate manifests templates-generate add-license projectsveltos-crds

.PHONY: projectsveltos-crds
projectsveltos-crds: yq
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
projectsveltos-crds: yq
projectsveltos-crds: $(YQ)

Copy link
Contributor Author

@BROngineer BROngineer Feb 5, 2025

Choose a reason for hiding this comment

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

why replace yq target with $(YQ)?

Makefile Outdated Show resolved Hide resolved
@BROngineer BROngineer requested a review from eromanova February 5, 2025 13:42
@BROngineer BROngineer force-pushed the chore/issue-1040-maintain-sveltos-crds-version branch from 30dcfbd to 74ccc13 Compare February 5, 2025 15:01
@BROngineer BROngineer force-pushed the chore/issue-1040-maintain-sveltos-crds-version branch from 74ccc13 to a975af1 Compare February 5, 2025 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

[chore] Add Makefile target to keep projectsveltos CRDs consistent with projectsveltos release version
2 participants