Repository navigation
chore: update in-repo references for the move to brawer/osmdiffs - #802
Merged
Merged
Conversation
The repo moved from github.com/alltheplaces/osm-diffs to
github.com/brawer/osmdiffs (new owner, hyphen dropped). This updates
every in-repo reference to the old path:
- Functional / output-affecting:
- ghcr.io image name: Containerfile ARG default + LABEL,
scripts/sbom/generate-sbom.sh default. (release-build.yml already
derives it from ${{ github.repository }}.)
- provenance BOM: REPO_URL and the purl in src/pipeline/provenance.rs
and the matching purl in scripts/sbom/pipeline.jq.
- datapackage manifest: HOMEPAGE, and NAME "osm-diffs" -> "osmdiffs"
(the Frictionless package slug; no prod run has frozen it yet, so
now is the cost-free moment to match the new repo). Golden file and
the two name assertions updated to match.
- scripts/test-on-hetzner: DEFAULT_REPO clone URL and the image-ref
example / sanitisation test.
- Documentation and code-comment links, issue/PR shorthands, README
badges (Coveralls, OpenSSF Scorecard).
Deliberately left unchanged:
- The frozen UUIDv5 seeds in provenance.rs (SERIAL_NAMESPACE) and
datapackage.rs (ID): the constants were minted from the old URL and
must never be regenerated, so the derivation comments keep the old
URL verbatim, now with a note saying why.
- The Rust crate / binary name (Cargo.toml `name = "osm-diffs"`) and
everything downstream of it (binary path, systemd unit names, local
workdir names, `cargo_bin!`, log process name) -- a separate decision.
- `https://github.com/alltheplaces/` as the "All The Places" data
supplier in the SBOM: that is genuinely the ATP org, not this repo.
Full test suite (cargo + the Hetzner script pytest suite) passes.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01ADxF1ZE2j9Ki51fiJQQLVo
Stewardship of this pipeline is now personal (the repo left the
alltheplaces org; the author runs and pays for production). The SBOM
`supplier` field -- "who stands behind this software" -- should say so.
- `provenance.rs`: `supplier()` (the BOM's `metadata.supplier` and its
output component) now returns `{Sascha Brawer, https://brawer.ch}`.
Split out `atp_supplier()` for the AllThePlaces *input* data
component, which is genuinely supplied by the ATP project and keeps
`{All The Places, https://github.com/alltheplaces/}` -- same shape as
the existing `osm_supplier()`.
- `scripts/sbom/{merge,pipeline,tippecanoe-binary}.jq`: the matching
`metadata.supplier` / `metadata.component.supplier` in the
container-image SBOM, kept in sync by grep as the comments note.
- Per-component suppliers that are correctly third parties (Felt for
tippecanoe, Alpine for apk-derived tools, the OSMF for the planet
dump) are unchanged.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01ADxF1ZE2j9Ki51fiJQQLVo
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to the GitHub transfer + rename (
alltheplaces/osm-diffs→brawer/osmdiffs). Updates every in-repo reference to the old path.Functional / output-affecting
ContainerfileARG default +org.opencontainers.image.descriptionLABEL,scripts/sbom/generate-sbom.shfallback default. (release-build.ymlalready uses${{ github.repository }}.)REPO_URL+ purl insrc/pipeline/provenance.rs, and the matchingpkg:github/...purl inscripts/sbom/pipeline.jq.src/pipeline/datapackage.rs):HOMEPAGE, andNAME"osm-diffs"→"osmdiffs"— the Frictionless package slug. No production run has published (and thus frozen) it yet, so this is the cost-free moment to align it with the new repo. Golden file + the twonameassertions updated.scripts/test-on-hetzner:DEFAULT_REPOclone URL, the--imageexample, and the label-sanitisation test.supplier: changed from the All The Places org to{Sascha Brawer, https://brawer.ch}inprovenance.rsandscripts/sbom/{merge,pipeline,tippecanoe-binary}.jq— stewardship of this pipeline is now personal. Split outatp_supplier()so the AllThePlaces input data component still names All The Places as its supplier; third-party per-component suppliers (Felt, Alpine, OSMF) are untouched.Non-functional
Doc prose links, code-comment issue/PR shorthands, README badges (Coveralls + OpenSSF Scorecard). Issue/PR numbers are preserved across a GitHub transfer, so
brawer/osmdiffs#NNNresolves to the same items; old links redirect regardless.Deliberately left unchanged
SERIAL_NAMESPACE(provenance) andID(datapackage). These constants were minted from the old URL and must never be regenerated; the derivation comments keep the old URL verbatim, now with a note explaining why.Cargo.tomlname = "osm-diffs") and everything downstream (binary path, systemd unit names, local workdir dirs,cargo_bin!, log process name) — a separate decision for later.https://github.com/alltheplaces/alltheplacesas the ATP source project, and the ATP org as supplier of the input scraped-POI dataset — those are genuinely All The Places, not this repo.Testing
cargo test(263 lib + 6 integration) and the Hetzner script pytest suite (81) all pass.cargo fmt+clippyclean (the 3 pre-existingchunks_exactwarnings are unrelated).🤖 Generated with Claude Code
https://claude.ai/code/session_01ADxF1ZE2j9Ki51fiJQQLVo