Developers should use npm ci when installing the dashboard from source
#277
Assignees
Comments
a-roberts
changed the title
mnuttall
changed the title
npm ci when installing the dashboard from source
|
/assign |
|
PR: #317 |
|
/close |
|
@jessm12: Closing this issue. In response to this:
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. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Expected Behavior
package-lock.json should be adhered to
Actual Behavior
We suggest
npm installing. We should suggest Node.js 10.x I believe andnpm ciinstead 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
The text was updated successfully, but these errors were encountered: