Skip to content

Conversation

@lc-cn
Copy link
Contributor

@lc-cn lc-cn commented Dec 1, 2025

The site content is documentation of Robot Protocol Conversion Service Written in NodeJS

Copilot AI review requested due to automatic review settings December 1, 2025 00:42
Copilot finished reviewing on behalf of lc-cn December 1, 2025 00:43
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR requests a new JS.ORG subdomain "onebots.js.org" for a Robot Protocol Conversion Service documentation site written in NodeJS. The subdomain will point to lc-cn.github.io/onebots.

Key changes:

  • Adds a new subdomain entry "onebots" to the active CNAME configuration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@lc-cn
Copy link
Contributor Author

lc-cn commented Dec 1, 2025

@copilot I think it's ok,bug CI validate Error with
Resuming validateCNAMEsFile process Line 2277: Expected to find ' "onebots": "lc-cn.github.io/onebots",' after existing line Error: Expected to find "onebots": "lc-cn.github.io/onebots",after existing line Line 2280: Expected no line, but found ' "onebots": "lc-cn.github.io/onebots",' Error: Expected no line, but found "onebots": "lc-cn.github.io/onebots", Validation failed for validateCNAMEsFile Error: Process completed with exit code 1.

Copy link
Member

@MattIPv4 MattIPv4 left a comment

Choose a reason for hiding this comment

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

Copilot and the CI error are correct, it is not a bug...

❌ Your requested js.org subdomain must be inserted in alphabetical order in the cnames_active.js file before we can process your request. Please fix this. 🔤

one sorts before ong

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants