data(providers): fill missing contact and social fields (batch 2) - #2636
data(providers): fill missing contact and social fields (batch 2)#2636EazyHood wants to merge 1 commit into
Conversation
Summary
Errors per inputErrors in ./references/providers/providers.csv
|
USS-Supervisor
left a comment
There was a problem hiding this comment.
Verdict: REQUEST_CHANGES
Risk: HIGH
Summary: The provider CSV still parses and required validation passed, but this PR introduces destructive provider-row churn outside the stated contact/social fill scope.
Findings:
- HIGH
references/providers/providers.csvrows 150, 259, 262, and 456: the PR removes the canonical provider rows forcidgravity,fil-one,filecoin-onchain-cloud, andpalmerathat are present on currentmain. Please rebase/restore these provider records so the PR only fills empty provider metadata cells.
Confidence: HIGH
1286400 to
fdd9ef0
Compare
|
Fixed. The branch is now rebased onto current main, so the four provider rows that were reported as The flagged social handles have been cleared rather than guessed. I also re-checked every value this |
bde6fdd to
6af7659
Compare
|
@eugene17kotov This PR has passing required CI and is waiting on reviewer action. Could you please take a look when you have a chance? |
6af7659 to
a3d1962
Compare
a3d1962 to
e7b0818
Compare
|
The changes requested here have been addressed in the latest push. Could you take another look when you get a chance? |
e7b0818 to
13b99cc
Compare
Fills empty contact and social fields in
references/providers/providers.csvfrom each provider'sown site and documentation. No existing values were changed; only empty cells were populated.
Rewards address
0x06f44f4839fd5df4f4670036d028b29dec939363