Skip to content

chore: stop tracking compiled JS output; fix deploy pipeline trigger - #48

Merged
Joe-Ortiz merged 2 commits into
mainfrom
copilot/fix-conflicting-map-files
Mar 21, 2026
Merged

chore: stop tracking compiled JS output; fix deploy pipeline trigger#48
Joe-Ortiz merged 2 commits into
mainfrom
copilot/fix-conflicting-map-files

Conversation

Copilot AI commented Mar 21, 2026

Copy link
Copy Markdown
Contributor

Compiled js/ artifacts were committed to the repo, causing repeated merge conflicts across concurrent PRs. Since deploy.yml already runs npm run build before staging files, the committed output was purely redundant.

Changes

  • .gitignore — Replaced three granular patterns (js/*.js, js/*.js.map, js/*.d.ts) with a single js/ exclusion. The old patterns also missed *.d.ts.map files, which were being tracked.
  • git rm --cached — Evicted the four already-tracked files (app.js, storage.js, app.d.ts.map, storage.d.ts.map) from the index without touching the working tree.
  • update-cache-version.yml — Swapped the path trigger from js/**src/** so the service-worker update → deploy chain still fires on TypeScript source changes now that js/ is no longer committed.
# before
paths:
  - 'js/**'

# after
paths:
  - 'src/**'
Original prompt

The .map files keep causing conflicts when I have multiple PRs open and the .js is redundant. My app is published with GitHub Pages and uses the #file:deploy.yml Is there a way to compile the ts files and make sure the .js files are available in prod?

The user has attached the following file paths as relevant context:

  • .github\workflows\deploy.yml

Created from VS Code.


⚡ Quickly spin up Copilot coding agent tasks from anywhere on your macOS or Windows machine with Raycast.

Copilot AI changed the title [WIP] Fix conflicts with .map files and ensure .js availability in prod chore: stop tracking compiled JS output; fix deploy pipeline trigger Mar 21, 2026
Copilot AI requested a review from Joe-Ortiz March 21, 2026 01:08
@Joe-Ortiz
Joe-Ortiz marked this pull request as ready for review March 21, 2026 01:13
@Joe-Ortiz
Joe-Ortiz merged commit 395f4cc into main Mar 21, 2026
1 check passed
@Joe-Ortiz
Joe-Ortiz deleted the copilot/fix-conflicting-map-files branch March 21, 2026 01:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants