Skip to content

Create opsight.yaml#833

Merged
ccerv1 merged 2 commits intoopensource-observer:mainfrom
bytemaster333:main
Jan 20, 2026
Merged

Create opsight.yaml#833
ccerv1 merged 2 commits intoopensource-observer:mainfrom
bytemaster333:main

Conversation

@bytemaster333
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Member

@ccerv1 ccerv1 left a comment

Choose a reason for hiding this comment

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

There's nothing in this GitHub, just a readme and a funding.json

@bytemaster333
Copy link
Copy Markdown
Contributor Author

There's nothing in this GitHub, just a readme and a funding.json

Similarities of this project have been made to the Arbitrum and Stellar ecosystems before. I'm thinking of adapting it in optimism. So I applied as an idea. I haven't moved on to the construction phase of the project yet.

@ccerv1
Copy link
Copy Markdown
Member

ccerv1 commented Jan 17, 2026

Issues found:

  1. File location: The file should be in a subdirectory matching the first character of the project name.

    • Current: data/projects/opsight.yaml
    • Correct: data/projects/o/opsight-bytemaster333.yaml
  2. Incorrect project name: Since this is a single repository (not a GitHub organization), the name should follow the pattern [repo-name]-[owner-name]:

    • Current: name: OPSight
    • Correct: name: opsight-bytemaster333

    The display_name can remain OPSight.

See naming documentation: https://docs.oso.xyz/docs/projects/#give-your-project-a-unique-name-slug

Please fix these issues and update your PR.

@ccerv1 ccerv1 added the pr:critical_issue PR has critical issue preventing approval label Jan 17, 2026
@ccerv1
Copy link
Copy Markdown
Member

ccerv1 commented Jan 20, 2026

/validate a09fede

@oso-prs
Copy link
Copy Markdown

oso-prs Bot commented Jan 20, 2026

Validation Results

⛔ Found 4 errors ⛔

commit a09fede0b28a218f856e33f2ee8ccc7941b6cad4


Overview

https://defillama.com/protocol/morpho-compound

  • ❌ https://defillama.com/protocol/morpho-compound is not a valid DefiLlama slug

  • ✅ https://defillama.com/protocol/morpho-compound is a valid DefiLlama URL

https://defillama.com/protocol/robinhood

  • ✅ https://defillama.com/protocol/robinhood is a valid DefiLlama URL

  • ✅ https://defillama.com/protocol/robinhood is a valid DefiLlama slug

https://defillama.com/protocol/morpho-aavev3

  • ❌ https://defillama.com/protocol/morpho-aavev3 is not a valid DefiLlama slug

  • ✅ https://defillama.com/protocol/morpho-aavev3 is a valid DefiLlama URL

https://defillama.com/protocol/world-chain

  • ✅ https://defillama.com/protocol/world-chain is a valid DefiLlama URL

  • ✅ https://defillama.com/protocol/world-chain is a valid DefiLlama slug

https://defillama.com/protocol/morpho-aave

  • ❌ https://defillama.com/protocol/morpho-aave is not a valid DefiLlama slug

  • ✅ https://defillama.com/protocol/morpho-aave is a valid DefiLlama URL

https://defillama.com/protocol/morpho-blue

  • ❌ https://defillama.com/protocol/morpho-blue is not a valid DefiLlama slug

  • ✅ https://defillama.com/protocol/morpho-blue is a valid DefiLlama URL

@ccerv1 ccerv1 merged commit 16ec641 into opensource-observer:main Jan 20, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:critical_issue PR has critical issue preventing approval

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants