Skip to content

commet: Add version 0.4.2#18110

Open
arvdk wants to merge 1 commit into
ScoopInstaller:masterfrom
arvdk:commet
Open

commet: Add version 0.4.2#18110
arvdk wants to merge 1 commit into
ScoopInstaller:masterfrom
arvdk:commet

Conversation

@arvdk

@arvdk arvdk commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Closes #17618

  • Use conventional PR title: <manifest-name[@version]|chore>: <general summary of the pull request>
  • I have read the Contributing Guide

@coderabbitai

coderabbitai Bot commented Jun 21, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

A new Scoop manifest file bucket/commet.json is added for Commet, a Matrix client. The manifest declares version 0.4.2, a description, homepage (https://commet.chat/), and GPL-3.0-or-later license. It specifies a 64-bit Windows ZIP download URL from the Commet GitHub releases with its SHA256 hash, sets the extraction directory to Release, creates a Windows shortcut from commet.exe to Commet, and configures GitHub-based version checking and autoupdate with a $version-templated download URL.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the main change: adding Commet version 0.4.2 manifest to the repository, matching the file and manifest version added.
Description check ✅ Passed The PR description includes the required checklist items completed and references the linked issue #17618, meeting the repository template requirements.
Linked Issues check ✅ Passed The PR adds Commet 0.4.2 manifest satisfying the request in #17618 for adding Commet to Scoop, meeting all criteria: well-known, English interface, full version, and standard installation methods.
Out of Scope Changes check ✅ Passed The PR only adds the Commet 0.4.2 manifest file, which is directly in scope with the linked issue requesting Commet package inclusion.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai 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.

🧹 Nitpick comments (1)
bucket/commet.json (1)

1-29: 🧹 Nitpick | 🔵 Trivial

Run the standard Scoop manifest validation flow before merge.

Use:

  • scoop config debug true
  • scoop config gh_token <your-github-token>
  • .\bin\checkver.ps1 -App commet -f
  • .\bin\formatjson.ps1 -App commet
  • scoop install bucket/commet.json -a 64bit

If any rule conflicts come up, use the Contribution Guide/Wiki for resolution.

As per coding guidelines, local checkver/format/install verification should be provided for Scoop manifests, with references to the official Contribution Guide and Wiki.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@bucket/commet.json` around lines 1 - 29, The Scoop manifest file
(commet.json) has not been validated using the standard Scoop validation
workflow before submission. Configure Scoop with debug mode enabled and a valid
GitHub token, then run the checkver.ps1 script against the commet app to verify
the manifest version and download URL are correct, followed by formatjson.ps1 to
ensure proper JSON formatting, and finally perform a test installation using
scoop install to validate the manifest works correctly. If any validation errors
or rule conflicts arise, consult the official Scoop Contribution Guide and Wiki
for resolution guidance.

Source: Coding guidelines

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@bucket/commet.json`:
- Around line 1-29: The Scoop manifest file (commet.json) has not been validated
using the standard Scoop validation workflow before submission. Configure Scoop
with debug mode enabled and a valid GitHub token, then run the checkver.ps1
script against the commet app to verify the manifest version and download URL
are correct, followed by formatjson.ps1 to ensure proper JSON formatting, and
finally perform a test installation using scoop install to validate the manifest
works correctly. If any validation errors or rule conflicts arise, consult the
official Scoop Contribution Guide and Wiki for resolution guidance.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 8c932555-280c-4931-8707-d134fc0df261

📥 Commits

Reviewing files that changed from the base of the PR and between 2fa69b0 and 0df3464.

📒 Files selected for processing (1)
  • bucket/commet.json

@arvdk

arvdk commented Jun 21, 2026

Copy link
Copy Markdown
Contributor Author

/verify

@github-actions

Copy link
Copy Markdown
Contributor

All changes look good.

Wait for review from human collaborators.

commet

  • Lint
  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

Check the full log for details.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Request]: Commet

1 participant