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

promoted git-sync-annotation to spec field #289

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

basit9958
Copy link
Contributor

@basit9958 basit9958 commented Apr 1, 2023

Ⅰ. Describe what this PR does

It promotes git-sync annotation to the spec field

II. Does this pull request fix one issue?

fixes #262

* validate helm chart job added

Signed-off-by: Basit Hasan <[email protected]>

* minor fix

Signed-off-by: Basit Hasan <[email protected]>

* add version in values.yaml

Signed-off-by: Basit Hasan <[email protected]>

---------

Signed-off-by: Basit Hasan <[email protected]>
Signed-off-by: Basit Hasan <[email protected]>
@basit9958 basit9958 force-pushed the Promote-git-sync-feature-as-the-spec-field branch from be56b59 to 17be123 Compare April 5, 2023 05:49
@basit9958 basit9958 marked this pull request as ready for review April 5, 2023 05:55
@basit9958
Copy link
Contributor Author

basit9958 commented Apr 5, 2023

cc @SimonCqk ci and e2e are failing because workflow runner is not picking the job. I have opened a pr to fix that please merge and re trigger the workflow here

@basit9958 basit9958 requested a review from SimonCqk April 5, 2023 16:39
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.

[feature request] Promote git sync feature as the spec field
2 participants