Skip to content

feat: 王者插件改为 npm 安装 - #81

Open
cchanlan wants to merge 1 commit into
KarinJS:mainfrom
cchanlan:main
Open

cchanlan wants to merge 1 commit into
KarinJS:mainfrom
cchanlan:main

Conversation

@cchanlan

@cchanlan cchanlan commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

变更内容

验证

  • 本地运行 node scripts/validate.js 通过。

@coderabbitai

coderabbitai Bot commented Nov 5, 2026

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 08f84928-e448-4752-b88a-e11e179615ed
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Tip

🎯 Customize your risk assessments

Guide how CodeRabbit assesses merge risk and security architecture risk. Highlight sensitive changes and explain your security boundaries using Merge Risk Instructions and Security Risk Instructions in your review settings. Security architecture risk assessments require the Advanced plan or above.

✅ Automatic approvals, with your rules

Let CodeRabbit approve pull requests once required threads are resolved, pre-merge checks pass, and your risk limits are met. Enable Automatic approvals in review settings, with overrides for teams or file paths.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@cchanlan

cchanlan commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor Author

CI 失败与本次条目修改无关:pr-check.yml 在 pull_request_target 下 checkout fork 代码,新版 actions/checkout@v4 默认 allow-unsafe-pr-checkout: false 会直接拒绝。

本地运行 node scripts/validate.js 已通过(33 条全部校验通过),本次仅将 karin-plugin-glory-of-kings 的 type 由 git 改为 npm。

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