Developers should use npm ci when installing the webhook extension from source

[a-roberts]

Expected Behavior

package-lock.json should be adhered to

Actual Behavior

We suggest npm installing. We should suggest Node.js 10.x I believe and npm ci instead so our package-lock.json is taken into account.

@dibbles noticed bold font styling and a scrollbar on the delete dialog for the extension when using Node 12 with a newly generated package-lock.json as a result of npm installing - by using npm ci with both the dashboard and extension, we're good.

See tektoncd/dashboard#277 for the related dashboard issue.

[Megan-Wright]

/assign

[Megan-Wright]

/close

[tekton-robot]

@Megan-Wright: Closing this issue.

In response to this:

/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.