Inconsistent Types for PipelineRef and TaskRef

[danielhelfand]

While working with the Tekton API as part of the cli project, I was adding validation to the tkn tr describe and tkn pr describe commands to check whether the properties are present on taskrun and pipelinerun specs. For taskruns, I used a nil check to determine if the TaskRef is present. For pipelineruns, I had to used an empty string for PipelineRef.

I am wondering why TaskRef is defined as follows here:

TaskRef *TaskRef json:"taskRef,omitempty"

And why a different approach was taken with PipelineRef here:

PipelineRef PipelineRef json:"pipelineRef,omitempty"

What was the thought behind using the * operator for TaskRef whereas all other Refs do not?

[danielhelfand]

/kind question

[vdemeester]

Will be solved by #1495 and #1497

[vdemeester]

Both pull-requests are merged, closing
/close

[tekton-robot]

@vdemeester: Closing this issue.

In response to this:

Both pull-requests are merged, closing
/close

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.