Skip to content

fix(export): untrack the 8 raw postcode export files#1492

Merged
dr5hn merged 1 commit into
masterfrom
fix/untrack-postcode-exports
Apr 28, 2026
Merged

fix(export): untrack the 8 raw postcode export files#1492
dr5hn merged 1 commit into
masterfrom
fix/untrack-postcode-exports

Conversation

@dr5hn
Copy link
Copy Markdown
Owner

@dr5hn dr5hn commented Apr 28, 2026

Follow-up to #1490. The gitignore entries added there don't apply because the files were already tracked at tiny placeholder sizes (committed during #1039 / #1403 initial exports wiring). The export workflow regenerates them at full size each run, exceeding GitHub's 100 MB hard limit on the PR push.

remote: error: File json/postcodes.json is 239.41 MB ... exceeds 100.00 MB
remote: error: File xml/postcodes.xml is 263.33 MB ... exceeds 100.00 MB
remote: error: File yml/postcodes.yml is 171.45 MB ... exceeds 100.00 MB
remote: error: File psql/postcodes.sql is 123.16 MB ... exceeds 100.00 MB
remote: error: File sqlite/postcodes.sqlite3 is 105.85 MB ... exceeds 100.00 MB

Untracking lets the existing gitignore (#1490) take effect. Compressed .gz versions continue to publish via GitHub Releases.

Refs: #1039, #1490

These were committed at tiny placeholder sizes during #1039's initial
exports wiring (#1403), but the export.yml workflow regenerates them
at full size every run — 239 MB json/postcodes.json, 263 MB
xml/postcodes.xml, 171 MB yml/postcodes.yml, 123 MB psql/postcodes.sql,
105 MB sqlite/postcodes.sqlite3, 90 MB sql/postcodes.sql, 73 MB
sqlserver/postcodes.sql — all over GitHub's 100 MB hard limit.

#1490 added .gitignore entries for the same paths but gitignore is
inert against tracked files, so the export PR's git push still failed.
Untrack here so the gitignore actually applies; large compressed
.gz versions continue to ship via GitHub Releases.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 28, 2026 06:45
@dosubot dosubot Bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Apr 28, 2026
@dr5hn dr5hn merged commit ee5abf7 into master Apr 28, 2026
1 check passed
@dr5hn dr5hn deleted the fix/untrack-postcode-exports branch April 28, 2026 06:45
@dosubot dosubot Bot added the exports label Apr 28, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes previously tracked raw postcodes.* export artifacts so the existing .gitignore rules (added in #1490) can actually take effect, preventing the export workflow from attempting to push >100MB files to GitHub.

Changes:

  • Deletes tracked placeholder versions of raw postcode exports across JSON/XML/YML/SQL/SQLite/CSV formats.
  • Ensures future workflow runs won’t re-add these large raw files to git (they remain generated + ignored; .gz versions continue to be publishable via Releases per #1490).

Reviewed changes

Copilot reviewed 6 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
yml/postcodes.yml Removes tracked raw YAML postcode export so it remains ignored going forward
xml/postcodes.xml Removes tracked raw XML postcode export so it remains ignored going forward
sqlserver/postcodes.sql Removes tracked raw SQL Server postcode export so it remains ignored going forward
sql/postcodes.sql Removes tracked raw MySQL postcode dump so it remains ignored going forward
psql/postcodes.sql Removes tracked raw PostgreSQL postcode dump so it remains ignored going forward
json/postcodes.json Removes tracked raw JSON postcode export so it remains ignored going forward
sqlite/postcodes.sqlite3 Removes tracked raw SQLite postcode export so it remains ignored going forward
csv/postcodes.csv Removes tracked raw CSV postcode export so it remains ignored going forward

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

exports size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants