Update rclone-raycast extension#26992
Conversation
|
Thank you for your first contribution! 🎉 🔔 @FTCHD you might want to have a look. You can use this guide to learn how to check out the Pull Request locally in order to test it. 📋 Quick checkout commandsBRANCH="ext/rclone-raycast"
FORK_URL="https://github.com/shtuseriff/raycast-extensions.git"
EXTENSION_NAME="rclone-raycast"
REPO_NAME="raycast-extensions"
git clone -n --depth=1 --filter=tree:0 -b $BRANCH $FORK_URL
cd $REPO_NAME
git sparse-checkout set --no-cone "extensions/$EXTENSION_NAME"
git checkout
cd "extensions/$EXTENSION_NAME"
npm install && npm run devWe're currently experiencing a high volume of incoming requests. As a result, the initial review may take up to 10-15 business days. |
Greptile SummaryAdds Confidence Score: 5/5Safe to merge — change is minimal, correct, and well-documented. All three files are clean: the code change uses a well-defined Node.js spawn option, the changelog follows Raycast conventions, and the package.json update is valid JSON. No bugs, security issues, or rule violations were found. No files require special attention.
|
| Filename | Overview |
|---|---|
| extensions/rclone-raycast/src/start-daemon.tsx | Adds windowsHide: true to spawn options — correct Node.js API usage to suppress the Windows console popup for the detached daemon process |
| extensions/rclone-raycast/package.json | Appends a contributors array with misterK; JSON remains valid and well-formed |
| extensions/rclone-raycast/CHANGELOG.md | New entry correctly placed at the top with {PR_MERGE_DATE} placeholder, describing the windowsHide change |
Reviews (1): Last reviewed commit: "Fix: Hide WinTerm window when running St..." | Re-trigger Greptile
pernielsentikaer
left a comment
There was a problem hiding this comment.
Looks good to me, approved 🔥
Description
Screencast
Checklist
npm run buildand tested this distribution build in Raycastassetsfolder are used by the extension itselfREADMEare placed outside of themetadatafolder