Update git-repos extension#26893
Conversation
- Merge branch \'contributions/merge-1775221914865\' - Pull contributions - Added windows compatibility
|
Thank you for your contribution! 🎉 🔔 @moored @nitishpdmn @go-dima @cameronsstone @hacdias @AaronMoat @shkreios 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/git-repos"
FORK_URL="https://github.com/NikolaiDokken/raycast-extensions.git"
EXTENSION_NAME="git-repos"
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. |
- Merge branch \'contributions/merge-1775221914865\' - Merge branch \'contributions/merge-1775221914865\'
Greptile SummaryThis PR adds Windows compatibility to the Key changes:
Confidence Score: 5/5This PR is safe to merge; the cross-platform refactoring is clean and all previously raised review concerns have been addressed. No P0 or P1 issues found. The glob-based scanning is functionally equivalent to the original find-based approach and includes dot:true to discover hidden directories. Path handling, browser detection, and type definitions are all correctly adapted for Windows. Previously flagged issues (dot:true, worktree deduplication comment, Preferences type) are resolved. No files require special attention. Important Files Changed
Greploops — Automatically fix all review issues by running Reviews (3): Last reviewed commit: "Use non-blocking glob instead of globSyn..." | Re-trigger Greptile |
- Merge branch \'contributions/merge-1775221914865\' - Merge branch \'contributions/merge-1775221914865\'
|
@greptileai do another review |
|
@greptileai do another review |
| onAction={() => { | ||
| // checking for app != null to not open in default app | ||
| function openIn(application?: Application) { | ||
| if (application?.bundleId) { |
There was a problem hiding this comment.
Is this a breaking change for mac?
There was a problem hiding this comment.
No @AaronMoat, the API reference specifies that open takes either a string (app identifier) or Application type.
pernielsentikaer
left a comment
There was a problem hiding this comment.
Looks good to me, approved 🔥
|
Published to the Raycast Store: |
|
🎉 🎉 🎉 We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag. |
Description
Added windows compatibility for the extension.
Screencast
Checklist
npm run buildand tested this distribution build in Raycastassetsfolder are used by the extension itselfREADMEare placed outside of themetadatafolder