Merge pull request #11972 from github/repo-sync

repo sync

Author: Octomerger

.github/workflows/js-lint.yml

@@ -10,6 +10,15 @@ on:
     branches:
       - main
   pull_request:
+    paths:
+      - '**.js'
+      - '**.mjs'
+      - '**.ts'
+      - '**.tsx'
+      # In case something like eslint or tsc or prettier upgrades
+      - 'package-lock.json'
+      # Ultimately, for debugging this workflow itself
+      - .github/workflows/js-lint.yml
 
 jobs:
   lint:

Dockerfile

@@ -50,8 +50,6 @@ COPY next.config.js ./next.config.js
 COPY tsconfig.json ./tsconfig.json
 COPY next-env.d.ts ./next-env.d.ts
 
-RUN npx tsc --noEmit
-
 RUN npm run build
 
 # --------------------------------------------------------------------------------
@@ -94,9 +92,10 @@ COPY --chown=node:node package*.json ./
 COPY --chown=node:node feature-flags.json ./
 COPY --chown=node:node next.config.js ./
 
-EXPOSE 80
-EXPOSE 443
-EXPOSE 4000
+# This makes sure server.mjs always picks up the preferred port
+ENV PORT=4000
+EXPOSE $PORT
+
 CMD ["node", "server.mjs"]