Skip to content

Show param default value in task describe#443

Merged
tekton-robot merged 2 commits intotektoncd:masterfrom
piyush-garg:desc_task
Nov 8, 2019
Merged

Show param default value in task describe#443
tekton-robot merged 2 commits intotektoncd:masterfrom
piyush-garg:desc_task

Conversation

@piyush-garg
Copy link
Copy Markdown
Contributor

@piyush-garg piyush-garg commented Nov 8, 2019

This will add the feature of showing default value
of params in task describe command
This will show the value on the same lines of
pipeline describe command

Submitter Checklist

These are the criteria that every PR should meet, please check them off as you
review them:

  • Includes tests (if functionality changed/added)
  • Includes docs (if user facing)
  • Regenerate the manpages and docs with make docs and make man if needed.
  • Run the code checkers with make check
  • Commit messages follow commit message best practices

See the contribution guide
for more details.

This will add the feature of showing default value
of params in task describe command
This will show the value on the same lines of
pipeline describe command
@tekton-robot tekton-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Nov 8, 2019
@piyush-garg
Copy link
Copy Markdown
Contributor Author

/cc pradeepitm12 danielhelfand

Comment thread pkg/cmd/task/describe.go Outdated
This will change DEFAULT_VALUE to DEFAULT VALUE
in pipeline and task describe command for
better user readability.

Also fixes test
@piyush-garg
Copy link
Copy Markdown
Contributor Author

/cc vdemeester hrishin

@vdemeester
Copy link
Copy Markdown
Member

vdemeester commented Nov 8, 2019

/cc @hrishin @vdemeester @pradeepitm12 @danielhelfand
(@piyush-garg you need to put the @ for it to work 😛 )

or not, nevermind 😹

Copy link
Copy Markdown
Member

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

/lgtm

@tekton-robot tekton-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 8, 2019
@pradeepitm12
Copy link
Copy Markdown
Contributor

/lgtm

Copy link
Copy Markdown
Member

@danielhelfand danielhelfand left a comment

Choose a reason for hiding this comment

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

/lgtm

@tekton-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: danielhelfand

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@tekton-robot tekton-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 8, 2019
@tekton-robot tekton-robot merged commit 1e372be into tektoncd:master Nov 8, 2019
chmouel added a commit that referenced this pull request Nov 22, 2019
#445 | [Chmouel Boudjnah] Fix brew install from tap | 2019/11/08-05:20
#446 | [Chmouel Boudjnah] Update README to 0.5.0 | 2019/11/08-07:01
#443 | [Piyush Garg] Show param default value in task describe | 2019/11/08-07:45
#443 | [Piyush Garg] Change DEFAULT_VALUE to DEFAULT VALUE | 2019/11/08-07:45
#447 | [Chmouel Boudjnah] Add Release process documentation | 2019/11/08-08:23
#449 | [Vincent Demeester] Linting yamls with yamllint 🏷 | 2019/11/08-10:09
#452 | [Piyush Garg] Add cloudEvent resource support to start command | 2019/11/11-22:56
#457 | [Piyush Garg] Fix resource creation failure in case of multiple | 2019/11/13-00:49
#450 | [danielhelfand] show status of steps with taskrun describe | 2019/11/13-08:00
#461 | [Piyush Garg] Update go.sum | 2019/11/14-03:01
null | [Vincent Demeester] Bump plumbing to latest changes 🔗 | 2019/11/14-04:36
null | [Dan Lorenc] Add a --filename parameter to 'tkn tasks start'. | 2019/11/15-02:38
null | [Piyush Garg] Fix interactive input params on pipeline start | 2019/11/15-03:54
null | [Daniel Helfand] include os and version in issue template | 2019/11/18-14:57
null | [Chmouel Boudjnah] Add Installtion instruction when compiling from source | 2019/11/20-06:23
null | [Navid Shaikh] Uses vendored deps while building the binary in Makefile | 2019/11/20-07:58
null | [Daniel Helfand] add check in pr logs for if condition available | 2019/11/21-02:53

Signed-off-by: Chmouel Boudjnah <chmouel@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cla: yes lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants