Update CI jobs to remove unnecessary npm config and use of setup-node

There's no need to change the npm prefix as we don't rely on global
package installs for any of the CI jobs.

Also remove `setup-node` usage from the build test job as it only
needs go. This cuts the time for the build job in half.

Author: AlanGreene

.github/workflows/ci.yml

@@ -42,14 +42,6 @@ jobs:
           cache-dependency-path: "${{ github.workspace }}/src/github.com/tektoncd/dashboard/go.sum"
           go-version-file: "${{ github.workspace }}/src/github.com/tektoncd/dashboard/go.mod"
 
-      - name: Set up Node.js
-        uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0
-        with:
-          cache: npm
-          cache-dependency-path: src/github.com/tektoncd/dashboard/package-lock.json
-          node-version-file: src/github.com/tektoncd/dashboard/.nvmrc
-          registry-url: 'https://registry.npmjs.org/'
-
       - name: Install dependencies
         run: |
           echo "${GOPATH}/bin" >> "$GITHUB_PATH"

test/presubmit-tests.sh

@@ -83,10 +83,6 @@ function get_node() {
 function node_npm_install() {
   local failed=0
   get_node
-  echo "Configuring npm"
-  mkdir ~/.npm-global
-  npm config set prefix '~/.npm-global'
-  export PATH=$PATH:$HOME/.npm-global/bin
   echo "Installing package dependencies"
   npm ci || failed=1 # similar to `npm install` but ensures all versions from lock file
   return ${failed}