Skip to content

chore(ci): update minAppVersion to 1.12.7 and add git plugin to release - #18

Merged
ClaudiaFang merged 1 commit into
masterfrom
claude/update-versions-cicd-AcNw1
Apr 26, 2026
Merged

ClaudiaFang merged 1 commit into
masterfrom
claude/update-versions-cicd-AcNw1

Conversation

@ClaudiaFang

Copy link
Copy Markdown
Member
  • Set manifest.json minAppVersion to 1.12.7
  • Add @semantic-release/git plugin so release commits versions.json and
    other updated files back to the repository

- Set manifest.json minAppVersion to 1.12.7
- Add @semantic-release/git plugin so release commits versions.json and
  other updated files back to the repository
@ClaudiaFang
ClaudiaFang merged commit 7f21cad into master Apr 26, 2026
17 checks passed
@ClaudiaFang
ClaudiaFang deleted the claude/update-versions-cicd-AcNw1 branch April 26, 2026 03:35

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request updates the minimum application version in manifest.json and configures the @semantic-release/git plugin in .releaserc.json to automate release commits. Feedback suggests including package-lock.json in the release assets to ensure it stays synchronized with package.json and shortening the release commit message by removing the full release notes to maintain a cleaner git history.

Comment thread .releaserc.json
[
"@semantic-release/git",
{
"assets": ["manifest.json", "versions.json", "CHANGELOG.md", "package.json"],

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

When defining a custom assets list for the @semantic-release/git plugin, it overrides the default set. It is highly recommended to include package-lock.json (or your project's equivalent lockfile) to ensure it stays in sync with the version bump in package.json during releases. This prevents the repository from having an out-of-date lockfile after an automated release.

Suggested change
"assets": ["manifest.json", "versions.json", "CHANGELOG.md", "package.json"],
"assets": ["manifest.json", "versions.json", "CHANGELOG.md", "package.json", "package-lock.json"],

Comment thread .releaserc.json
"@semantic-release/git",
{
"assets": ["manifest.json", "versions.json", "CHANGELOG.md", "package.json"],
"message": "chore(release): ${nextRelease.version} [skip ci]\n\n${nextRelease.notes}"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Including ${nextRelease.notes} in the commit message can lead to extremely large messages, especially for releases with many changes. This can clutter the git history and occasionally cause issues with git hooks or platform limits. Since the full release notes are already captured in CHANGELOG.md and on the GitHub release page, a more concise message is generally preferred.

Suggested change
"message": "chore(release): ${nextRelease.version} [skip ci]\n\n${nextRelease.notes}"
"message": "chore(release): ${nextRelease.version} [skip ci]"

@ClaudiaFang

Copy link
Copy Markdown
Member Author

🎉 This PR is included in version 1.0.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

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