Skip to content

Cointelegraph Decentralization Guardians#864

Open
oneanother wants to merge 2 commits intomonad-developers:mainfrom
oneanother:ctdg-mainnet-val-1
Open

Cointelegraph Decentralization Guardians#864
oneanother wants to merge 2 commits intomonad-developers:mainfrom
oneanother:ctdg-mainnet-val-1

Conversation

@oneanother
Copy link

added Cointelegraph's mainnet validator identity information

added Cointelegraph's mainnet validator identity information
@oneanother oneanother requested a review from a team as a code owner March 16, 2026 15:14
@greptile-apps
Copy link

greptile-apps bot commented Mar 16, 2026

Greptile Summary

Adds a new mainnet validator identity file for Cointelegraph Decentralization Guardians (validator ID 189). The JSON file follows the required schema with valid secp and bls key formats, and the filename correctly matches the secp key.

  • Schema fields and types match the example template
  • SECP key (66 chars) and BLS key (96 chars) are consistent with existing validators
  • File is missing a trailing newline (185/188 existing files have one)
  • description field is empty — only 1 other validator out of 188 has an empty description

Confidence Score: 4/5

  • This PR is safe to merge — it adds a single validator identity JSON file with correct schema and key formats.
  • Score of 4 reflects that the file conforms to the required schema and validation checks should pass, with only minor style issues (missing trailing newline and empty description). The on-chain key match cannot be verified in this review but is covered by CI.
  • No files require special attention.

Important Files Changed

Filename Overview
mainnet/027802cce1bdec2443c4ca67cc7384ccd5087d64acab480c77ce8eb3d515809b44.json New validator identity file for Cointelegraph Decentralization Guardians (ID 189). Schema and key formats are correct; minor issues: missing trailing newline and empty description field.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[PR Submitted: New Validator JSON] --> B{Schema Valid?}
    B -->|Yes| C{Filename matches SECP key?}
    B -->|No| X[Reject]
    C -->|Yes| D{On-chain keys match?}
    C -->|No| X
    D -->|Yes| E{Logo URL valid?}
    D -->|No| X
    E -->|Yes| F[Merge: Validator 189 Added]
    E -->|No| X
Loading

Last reviewed commit: b8d0968

"secp": "027802cce1bdec2443c4ca67cc7384ccd5087d64acab480c77ce8eb3d515809b44",
"bls": "8813f9992c00556c3ea129c9055dfc5d72227b983a94ba826306242f641971cc92afb286c5d22b987fe726a857bf84d7",
"website": "https://cointelegraph.com/decentralization-guardians/",
"description": "",
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Empty description field
The description field is empty. Nearly all other validators (187/188) provide a description of their organization. Consider adding a brief description of Cointelegraph Decentralization Guardians to help users identify the validator.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
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.

1 participant