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

ref(*): sign and package release charts #303

Merged
merged 1 commit into from
Dec 13, 2016

Conversation

vdice
Copy link
Member

@vdice vdice commented Dec 9, 2016

As opposed to signing (and modifying) release artifact at a later point in time.

TODO

@vdice vdice added this to the v2.10 milestone Dec 9, 2016
@vdice vdice self-assigned this Dec 9, 2016
@vdice vdice force-pushed the sign-and-package-release-charts branch 20 times, most recently from f80c6fd to ba5bcca Compare December 12, 2016 18:26
@vdice vdice requested a review from kmala December 12, 2016 21:19
@vdice vdice force-pushed the sign-and-package-release-charts branch 2 times, most recently from d3250f0 to 4b06c14 Compare December 12, 2016 21:45
vdice pushed a commit to vdice/workflow that referenced this pull request Dec 12, 2016
@vdice vdice force-pushed the sign-and-package-release-charts branch from 4b06c14 to 4b1f290 Compare December 12, 2016 21:52
@kmala
Copy link
Contributor

kmala commented Dec 12, 2016

i think the workflow-v2.9.0.tgz chart got added by mistake to the PR or is it used somewhere?

@kmala kmala added the LGTM1 label Dec 12, 2016
As opposed to signing (and modifying) release artifact at a later point in time.
@vdice vdice force-pushed the sign-and-package-release-charts branch from 4b1f290 to 6963659 Compare December 12, 2016 22:05
@vdice
Copy link
Member Author

vdice commented Dec 12, 2016

@kmala indeed, it was a mistake; now removed. Thanks!

@vdice vdice requested a review from mboersma December 12, 2016 22:06
@vdice vdice merged commit 6e4f49b into deis:master Dec 13, 2016
@vdice vdice deleted the sign-and-package-release-charts branch December 13, 2016 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants