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

Increase column width when listing releases #1627

Merged
merged 1 commit into from
Dec 5, 2016
Merged

Increase column width when listing releases #1627

merged 1 commit into from
Dec 5, 2016

Conversation

mortenlj
Copy link
Contributor

@mortenlj mortenlj commented Dec 2, 2016

(Second attempt with correct author.email)

After #1560 was merged, names of releases can now be up to 53 characters long, up from 14 previously. When listing releases, the maximum column width is 30 characters, meaning it's not possible to get the full name of releases using the full available length.

This change just increases the maximum column width to 60, to allow for longer release names.

PS.
I'm working on getting a CLA signed by my employer, even if it does seem a bit silly for such a small change. :)

@k8s-ci-robot
Copy link

Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please follow instructions at https://github.com/kubernetes/kubernetes/wiki/CLA-FAQ to sign the CLA.

Once you've signed, please reply here (e.g. "I signed it!") and we'll verify. Thanks.


If you have questions or suggestions related to this bot's behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow issue:) repository.

@k8s-reviewable
Copy link

This change is Reviewable

@k8s-ci-robot k8s-ci-robot added the cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. label Dec 2, 2016
@technosophos
Copy link
Member

Looks good. As soon as the LF CLA passes I will merge.

@mortenlj
Copy link
Contributor Author

mortenlj commented Dec 5, 2016

In theory, the CLA should now be in order. I guess it tries checking again after a comment..?

@mortenlj
Copy link
Contributor Author

mortenlj commented Dec 5, 2016

Or do I have to push a new commit?

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. and removed cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels Dec 5, 2016
@technosophos
Copy link
Member

Thank you! I'm merging this into 2.1 now.

@technosophos technosophos merged commit 5939639 into helm:master Dec 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants