Skip to content

Add teleport extension#8956

Merged
raycastbot merged 6 commits intoraycast:mainfrom
lamberttraccard:ext/teleport
Nov 1, 2023
Merged

Add teleport extension#8956
raycastbot merged 6 commits intoraycast:mainfrom
lamberttraccard:ext/teleport

Conversation

@lamberttraccard
Copy link
Contributor

@lamberttraccard lamberttraccard commented Oct 25, 2023

Description

Screencast

Checklist

@raycastbot
Copy link
Collaborator

Congratulation on your new Raycast extension! 🚀

We will review it shortly. Once the PR is approved and merged, the extension will be available on the Store.

@raycastbot raycastbot added the new extension Label for PRs with new extensions label Oct 25, 2023
Copy link
Collaborator

@pernielsentikaer pernielsentikaer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi 👋

Thanks for your contribution 💪

I have now tested your extension, and I have some feedback ready for you:

It would be better to see if you can use spawn instead of exec since it's possible to inject scripts

I'm looking forward to testing this extension again 🔥

Request a new review when you are ready. Feel free to contact me here or at Slack if you have any questions.

@pernielsentikaer pernielsentikaer self-assigned this Oct 30, 2023
@lamberttraccard
Copy link
Contributor Author

Hi there!

I have made the requested changes. I hope they meet your requirements. 😊

Copy link
Collaborator

@pernielsentikaer pernielsentikaer left a 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 🔥

@raycastbot raycastbot merged commit 1802300 into raycast:main Nov 1, 2023
@raycastbot
Copy link
Collaborator

Published to the Raycast Store:
https://raycast.com/lamberttraccard/teleport

@raycastbot
Copy link
Collaborator

🎉 🎉 🎉

We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag.

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

Labels

new extension Label for PRs with new extensions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants