You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The JSON importer writes a snapshot commit on a local legacy-import
branch of the data repo but does not push or merge — those are deliberate
operator steps.
To integrate an importer run into runtime data:
After running the importer locally:
```bash
cd /path/to/codeforphilly-data
git push origin legacy-import
```
Open a PR on `CodeForPhilly/codeforphilly-data` from `legacy-import` to
`main`. The PR diff will be massive on the first run (44k files) and
small on subsequent runs (just the records that changed upstream).
Review the diff:
Path-shape conflicts: the importer writes `projects/.toml`
while the runtime spec's gitsheets config uses `projects/.toml`
path templates. Resolve by renaming files at merge time, or by
accepting a one-time `.gitsheets/` config swap on the legacy-import
branch before the first merge. Document the chosen approach here for
the next time.
The JSON importer writes a snapshot commit on a local
legacy-importbranch of the data repo but does not push or merge — those are deliberate
operator steps.
To integrate an importer run into runtime data:
After running the importer locally:
```bash
cd /path/to/codeforphilly-data
git push origin legacy-import
```
Open a PR on `CodeForPhilly/codeforphilly-data` from `legacy-import` to
`main`. The PR diff will be massive on the first run (44k files) and
small on subsequent runs (just the records that changed upstream).
Review the diff:
while the runtime spec's gitsheets config uses `projects/.toml`
path templates. Resolve by renaming files at merge time, or by
accepting a one-time `.gitsheets/` config swap on the legacy-import
branch before the first merge. Document the chosen approach here for
the next time.
conflicts (per the test in feat(importer): JSON-based laddr import producing legacy-import branch snapshots #57's plan). Resolve manually.
Merge.
This is intentionally manual because the merge is a runtime-data-affecting
step that warrants human review at cutover time.
Surfaced during the `laddr-import-via-json` plan (PR #57).