Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

[stable/spark] Add icon, home and update description #591

Merged
merged 2 commits into from
Feb 13, 2017
Merged

[stable/spark] Add icon, home and update description #591

merged 2 commits into from
Feb 13, 2017

Conversation

migmartri
Copy link
Contributor

PR in the context of normalizing/improving metadata in the official charts. Refs #124 and helm/monocular#93

NOTE: I have not bumped the chartVersion attribute in order to avoid race conditions/conflicts with existing PRs. Maintainers should be able to change this value before the merge. Llet me know if you prefer that I update it instead.

cc/ @lachie83

@k8s-ci-robot
Copy link
Contributor

Hi @migmartri. Thanks for your PR.

I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with @k8s-bot ok to test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Feb 7, 2017
@prydonius prydonius modified the milestone: Milestone 2 Feb 9, 2017
@lachie83 lachie83 added code reviewed lgtm Indicates that a PR is ready to be merged. UX reviewed labels Feb 13, 2017
@lachie83
Copy link
Contributor

LGTM

@lachie83 lachie83 self-assigned this Feb 13, 2017
@mgoodness mgoodness merged commit 8cd91d4 into helm:master Feb 13, 2017
yanns pushed a commit to yanns/charts that referenced this pull request Jul 28, 2017
* [stable/spark] Add icon, home and update description
* version bump
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. code reviewed lgtm Indicates that a PR is ready to be merged. UX reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants