Skip to content

chore: gitignore parity migrate/extract output (avoid committing scraped store data) - #210

Merged
JonasJesus42 merged 1 commit into
mainfrom
chore-gitignore-migrate-output
Aug 17, 2026
Merged

chore: gitignore parity migrate/extract output (avoid committing scraped store data)#210
JonasJesus42 merged 1 commit into
mainfrom
chore-gitignore-migrate-output

Conversation

@JonasJesus42

@JonasJesus42 JonasJesus42 commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Safety fix. The default output dirs (./parity-migrate, ./parity-extract) contain a real store's scraped content/assets. They were not gitignored, so running parity migrate inside the repo could accidentally commit customer store data to this public repo.

Adds parity-migrate/, parity-migrate*/, parity-extract/, parity-extract*/ to .gitignore.

Note: verified the public repo history is clean — no scraped output ever reached any origin/* ref (it only existed in local Conductor session checkpoints + the local working dir, both since removed).

🤖 Generated with Claude Code


Summary by cubic

Ignore default parity migrate and parity extract output directories to prevent committing scraped store content and assets. Previously these dirs were not ignored; now .gitignore excludes them.

Impact

  • Adds: parity-migrate/, parity-migrate*/, parity-extract/, parity-extract*/.
  • No migration required; history is clean. If you must commit sample output, use a non-default path or add a negation rule.

Written for commit a92dd03. Summary will update on new commits.

Review in cubic

…ped store data)

The default output dirs (`./parity-migrate`, `./parity-extract`) contain a
real store's scraped content/assets. Ignore them so a `parity migrate` run
inside the repo can't accidentally commit customer store data to a public repo.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@JonasJesus42
JonasJesus42 merged commit 5d5eaaa into main Aug 17, 2026
1 check passed
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.

1 participant