Skip to content

switch to pinhead#10

Merged
dschep merged 1 commit intomainfrom
pinhead
Mar 30, 2026
Merged

switch to pinhead#10
dschep merged 1 commit intomainfrom
pinhead

Conversation

@dschep
Copy link
Copy Markdown
Member

@dschep dschep commented Mar 30, 2026

Pinhead is a giant map-focused iconset which inclues all of maki and the overwhelming majority of temaki(and certainly everything relevant to this repo). This PR switches the maki_temaki field to a pinhead field. Also updates the ultra query to match (and actually work, i'd broken it due to changes in ultra 🫣 )

Used waysidemapping/pinhead#105 with this little script:

import fs from "node:fs";
import { migrateName } from "@waysidemapping/pinhead-js";

const landmarks = JSON.parse(fs.readFileSync("landmarks.json"));

for (const l of landmarks) {
  const [iconset, name] = l.maki_temaki.split("/", 2);
  l.pinhead = migrateName(name, iconset);
  delete l.maki_temaki
}

fs.writeFileSync("landmarks.json", JSON.stringify(landmarks, null, 4))

@jacobwhall
Copy link
Copy Markdown
Member

I'm all in, please merge whenever you think this is ready

@dschep dschep merged commit bb55893 into main Mar 30, 2026
@dschep dschep deleted the pinhead branch March 30, 2026 23:47
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