Skip to content

Create players-online.yaml#837

Closed
Lawd-STAR wants to merge 1 commit intoopensource-observer:mainfrom
Lawd-STAR:patch-1
Closed

Create players-online.yaml#837
Lawd-STAR wants to merge 1 commit intoopensource-observer:mainfrom
Lawd-STAR:patch-1

Conversation

@Lawd-STAR
Copy link
Copy Markdown

An AtomicSwaps dApp supporting Inter BlockChain Communication.

produce by PLAYERS☆ONLINE

An AtomicSwaps dApp supporting Inter BlockChain Communication. 

produce by PLAYERS☆ONLINE
@ccerv1
Copy link
Copy Markdown
Member

ccerv1 commented Jan 17, 2026

Issues found:

  1. Incomplete YAML syntax: The file ends with an empty social: field with no content. Either remove the social: line or add valid social media links under it.

  2. Misleading display_name: The display_name is "iTunes.com" but the project name is "players-online". The display name should accurately reflect the project.

  3. Missing GitHub URLs: At least one github URL is strongly recommended for projects in the OSS Directory.

  4. Typo in description: "AtomicSawps" should be "AtomicSwaps"

  5. Invalid description format: Line 5 ("Produce by PLAYERS☆ONLINE") appears to be a continuation of the description but is on a separate line. It should be part of the description field:

    description: An AtomicSwaps dApp supporting Inter BlockChain Communication. Produced by PLAYERS☆ONLINE

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
@lorenz234
Copy link
Copy Markdown
Collaborator

closing out, no activity

@lorenz234 lorenz234 closed this Apr 1, 2026
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.

3 participants