Skip to content

Conversation

@CodFrm
Copy link
Owner

@CodFrm CodFrm commented Sep 14, 2025

No description provided.

@CodFrm CodFrm requested review from Copilot and removed request for Copilot September 14, 2025 05:38
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 updates the version numbers for ScriptCat from version 1.1.1 to version 1.2, preparing for a new release. The version update follows a consistent pattern across both the browser extension manifest and the package.json.

  • Updates extension manifest version from 1.1.1 to 1.2.0.1100
  • Updates package.json version from 1.1.1 to 1.2.0-beta

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
src/manifest.json Updates browser extension version to 1.2.0.1100
package.json Updates npm package version to 1.2.0-beta

"manifest_version": 3,
"name": "__MSG_scriptcat__",
"version": "1.1.1",
"version": "1.2.0.1100",
Copy link

Copilot AI Sep 14, 2025

Choose a reason for hiding this comment

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

版本号格式 '1.2.0.1100' 不符合语义化版本规范。浏览器扩展应使用标准的语义化版本格式,如 '1.2.0' 或 '1.2.0-beta.1100'。当前格式可能在某些浏览器商店中被拒绝。

Suggested change
"version": "1.2.0.1100",
"version": "1.2.0-beta.1100",

Copilot uses AI. Check for mistakes.
{
"name": "scriptcat",
"version": "1.1.1",
"version": "1.2.0-beta",
Copy link

Copilot AI Sep 14, 2025

Choose a reason for hiding this comment

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

package.json 和 manifest.json 中的版本格式不一致。package.json 使用 '1.2.0-beta' 而 manifest.json 使用 '1.2.0.1100'。建议统一版本格式以避免混淆,例如都使用 '1.2.0' 或在 package.json 中也包含构建号。

Suggested change
"version": "1.2.0-beta",
"version": "1.2.0.1100",

Copilot uses AI. Check for mistakes.
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.

2 participants