data(providers): fill 14 missing github organisations, matched by website domain (batch 1 of 1) - #3577
data(providers): fill 14 missing github organisations, matched by website domain (batch 1 of 1)#3577EazyHood wants to merge 7 commits into
Conversation
Each organisation found by name on GitHub and accepted only when the organisation profile's website matches the provider's recorded website domain.
Summary
|
USS-Supervisor
left a comment
There was a problem hiding this comment.
Verdict: COMMENT
Risk: LOW
Summary: The provider batch is limited to filling previously empty GitHub organization cells in references/providers/providers.csv.
Findings: None from static review. CSV width is stable, the changed github values are stored in the expected domain-relative format, and no unrelated files are touched. I am not approving this cycle because I could not verify a current-cycle link-check success before cleanup.
Confidence: MEDIUM
|
For the next review cycle: the current-cycle link-check on this head is run 33737239201, posted above at the same minute as the review — 49 links checked, 0 errors, 0 timeouts. https://github.com/Chain-Love/chain-love/actions/runs/33737239201 |
|
Hi @eugene17kotov - following up on this stale PR. Required CI is passing and I do not see a current creator-side blocker, so it looks ready for reviewer action when you have a chance. |
|
@eugene17kotov this PR appears ready for review and has been waiting more than 3 days. Please review it when you can. |
Cell-level resolution: every cell main filled is kept, and this branch's cells survive only where main left them empty.
Three-way resolution per cell against the merge-base: this branch contributes only the 15 github cells it adds, and every other cell keeps main's current bytes. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
github.com/airtm-official now returns 404, and the only Airtm organisation left on GitHub (Airtm-Inc) holds a single fork with no link from airtm.com, so there is no organisation to point at. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
@eugene17kotov This PR is stale, required CI is passing, and no creator-side blocker is present. Could you please review it when you have a chance? |
Summary
Fills 14 empty
githubcells across 14 providers inreferences/providers/providers.csv(cyvers…zerohash). No rows added or removed, and no cell that already had a value was changed.Type of change
Scope
Networks affected: global (provider identities, not listings)
Categories affected:
references/providersAdditional notes, additional context / screenshots:
How each organisation was found, and the rule that decides whether it is written. For every provider with a
websiteand an emptygithub, GitHub's user search was queried for organisations by the provider's name. A candidate was accepted only if the organisation profile's own website field resolves to the same registrable domain as the provider's recordedwebsite. A name match alone was never enough; where no candidate carried the provider's domain, the cell was left empty. That is why this fills 14 cells and not the ~250 that are empty.Accuracy, measured before writing anything. The same procedure was run against 40 providers whose
githubis already recorded, chosen at random:So on the cells it decides, the rule agrees with the existing data 95.8% of the time, and the one disagreement is a second organisation of the same provider rather than a wrong provider.
Link checking. Every link in every touched row — the existing
website,docs,x,discordandtelegramvalues, plus the new organisation via the GitHub API — was requested with TLS verification on and compared against the accept list this repo's link-check uses (200,202,204,400,401,403,405,429). Rows carrying any link that fails today were left out entirely, since the check reads whole rows. Rows with a pre-existinglinkedinvalue were also left out (#3347).No overlap with other open PRs. Rows touched by #3250, #3323, #3324, #3325 and #3326 were excluded up front, so none of these cells is claimed twice and whichever lands first does not conflict with this one.
Changed on 24 Sep. The
airtmcell is gone:github.com/airtm-official, the organisation it pointed at, now returns 404, and the only Airtm organisation left on GitHub (Airtm-Inc) holds a single fork and is not linked from airtm.com, so there is nothing to point at.Style Guide conformance.
githubis written as the organisation login only (value after domain), matching the 466 existing values. Row order is untouched, so the file stays A–Z byslug.Links
Validation checklist
!providersyntax, and that entities in/networkssub-folders inherits records from/providersfolderOptional