Skip to content

altsendme@0.6.0: fix shortcut, update license field#18323

Open
BlackWinnerYoshi wants to merge 1 commit into
ScoopInstaller:masterfrom
BlackWinnerYoshi:altsendme-fixshortcut
Open

altsendme@0.6.0: fix shortcut, update license field#18323
BlackWinnerYoshi wants to merge 1 commit into
ScoopInstaller:masterfrom
BlackWinnerYoshi:altsendme-fixshortcut

Conversation

@BlackWinnerYoshi

Copy link
Copy Markdown
Contributor

Closes #18322

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

@coderabbitai

coderabbitai Bot commented Jul 16, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 51c1ea71-099a-4434-8c64-f2cb0e107386

📥 Commits

Reviewing files that changed from the base of the PR and between 5866b27 and 43e0970.

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

📝 Walkthrough

Walkthrough

The AltSendme manifest updates its description and homepage, changes the license from a string to an object containing an identifier and URL, and replaces both Windows shortcut entries with updated executable and display names.

Estimated code review effort: 1 (Trivial) | ~3 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning The manifest also changes description, homepage, and license metadata, which are unrelated to the linked shortcut bug. Split the metadata-only updates into a separate PR or issue, and keep this change focused on the shortcut fix.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title follows the required convention and summarizes the shortcut fix with the license update.
Description check ✅ Passed The PR description includes the linked issue and both required checklist items.
Linked Issues check ✅ Passed The shortcut entries now reference AltSendme.exe, matching the renamed binary and addressing issue #18322.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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.

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.

[Bug]: altsendme@0.6.0: Creating shortcut failed: couldn't find file (binary renamed)

1 participant