Inconsistent types for Header values extracted from bindings

[dibyom]

If I use $(header), the values are an array of strings. If I use $(header.key), the values are string.

I noticed this when working on #241. I think using strings instead of arrays would be the more expected behavior. This is also the behavior we use in most cases.

[bigkevmcd]

Possibly a good first task?

Affected tests are https://github.com/tektoncd/triggers/blob/master/pkg/template/event.go#L151 and https://github.com/tektoncd/triggers/blob/master/pkg/template/event_test.go#L597

[dibyom]

Hey @bigkevmcd I think #241 should fix this actually

[tekton-robot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.

/lifecycle stale

Send feedback to tektoncd/plumbing.

[tekton-robot]

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.

/lifecycle rotten

Send feedback to tektoncd/plumbing.

[tekton-robot]

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.

/close

Send feedback to tektoncd/plumbing.

[tekton-robot]

@tekton-robot: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.

/close

Send feedback to tektoncd/plumbing.

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.