data(providers): fill 15 missing github organisations, matched by website domain (batch 1 of 1) - #3577
Open
EazyHood wants to merge 1 commit into
Open
data(providers): fill 15 missing github organisations, matched by website domain (batch 1 of 1)#3577EazyHood wants to merge 1 commit into
EazyHood wants to merge 1 commit 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
Member
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
Contributor
Author
|
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 |
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.
Summary
Fills 15 empty
githubcells across 15 providers inreferences/providers/providers.csv(airtm…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 15 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.
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