-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Add geoping extension #18957
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add geoping extension #18957
Conversation
- changes - add screenshot
Congratulations on your new Raycast extension! 🚀 You can expect an initial review within five business days. Once the PR is approved and merged, the extension will be available on our Store. |
The last time I made a PR (#16623) I had forgotten to add a screenshot and @pernielsentikaer had made a few changes to it, unfortunately when I tried making changes after a few months to add the screenshot I ran into errors with $ npx @raycast/api@latest pull-contributions
ready - getting fork
ready - preparing clone
error - pulling new contributions
failed running git Command failed: git pull origin ext/geoping --ff-only
From https://github.com/Chiroyce1/raycast-extensions
* branch ext/geoping -> FETCH_HEAD
fatal: Not possible to fast-forward, aborting.
From https://github.com/Chiroyce1/raycast-extensions
* branch ext/geoping -> FETCH_HEAD
fatal: Not possible to fast-forward, aborting. Sorry about that - thats why this is an entirely new PR for the same extension, along with the screenshot. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
This PR adds a new extension "geoping" that pings global servers using Tailscale's DERP network to display latency information with visual indicators.
- The README.md is too minimal and should include more details about the extension's functionality, usage instructions, and its reliance on Tailscale's DERP network
- Since this is labeled as a new extension with a view command, it needs a
metadata
folder with screenshots as per Raycast Documentation - The
ping
function insrc/ping.tsx
should be wrapped in a try-catch block for better error handling - The List component should use
isLoading
prop to avoid empty state flicker as per Raycast Documentation - Consider using
showFailureToast
from@raycast/utils
instead of manual error handling in the catch block of the refresh function
💡 (2/5) Greptile learns from your feedback when you react with 👍/👎!
10 file(s) reviewed, 3 comment(s)
Edit PR Review Bot Settings | Greptile
edit: I managed to get it working by deleting the Terminal output$ tree -I node_modules
.
├── assets
│ └── extension_icon.png
├── CHANGELOG.md
├── metadata
│ └── geoping-1.png
├── package-lock.json
├── package.json
├── raycast-env.d.ts
├── README.md
├── src
│ ├── ping.tsx
│ ├── servers.ts
│ └── types.ts
└── tsconfig.json
4 directories, 11 files
$ git log -1
commit 4efe435d601617fa7323db036e1737a0e2bb2460 (HEAD -> main, tag: __raycast_latest_publish_ext/geoping__)
Author: Chiroyce <[email protected]>
Date: Tue May 6 14:19:40 2025 +0530
implement requested changes
$ git status
On branch main
nothing to commit, working tree clean
$ npx @raycast/api@latest pull-contributions
ready - getting fork
ready - preparing clone
ready - pulling new contributions
no new contributions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi 👋
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
An extension that shows your ping to servers around the world (Tailscale's derp network is used)
Screencast
Checklist
npm run build
and tested this distribution build in Raycastassets
folder are used by the extension itselfREADME
are placed outside of themetadata
folder