Skip to content
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

Added scripts that create and open blank Swift Playground project #465

Merged
merged 6 commits into from Jun 14, 2021

Conversation

qeude
Copy link
Contributor

@qeude qeude commented Jun 8, 2021

Description

Scripts that use playground cli to create and open new Playground project.
The first script opens a basic Playground.
The second one opens a view prototyping Playground.

Type of change

  • New script command

Screenshot

basic

view

Dependencies / Requirements

Requiring installing playground cli before.

Checklist

Copy link
Collaborator

@unnamedd unnamedd left a comment

Choose a reason for hiding this comment

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

Nice addition this pull request @qeude!

However, I strongly suggest you to move it to a new folder, maybe called swift-playground or just playground inside of apps folder. As you have more than one Script Command, I do think it will fit better there, even though it's a developer utility.

commands/developer-utils/open-new-playground.py Outdated Show resolved Hide resolved
commands/developer-utils/open-new-playground.py Outdated Show resolved Hide resolved
commands/developer-utils/open-new-playground.py Outdated Show resolved Hide resolved
commands/developer-utils/open-new-playground.py Outdated Show resolved Hide resolved
commands/developer-utils/open-new-playground.py Outdated Show resolved Hide resolved
commands/developer-utils/open-new-view-playground.py Outdated Show resolved Hide resolved
commands/developer-utils/open-new-view-playground.py Outdated Show resolved Hide resolved
commands/developer-utils/open-new-view-playground.py Outdated Show resolved Hide resolved
@qeude
Copy link
Contributor Author

qeude commented Jun 13, 2021

@unnamedd Thanks for those feedbacks, I did make the changes 😄

@fe9lix
Copy link
Member

fe9lix commented Jun 14, 2021

Hey @qeude! I think this got by mistake into the root folder in commands now. The idea was to move it into the apps folder, correct?

@qeude
Copy link
Contributor Author

qeude commented Jun 14, 2021

@fe9lix My bad, thought the idea was to put the playground folder in the commands, I'll fix this

@fe9lix fe9lix merged commit 2874c3d into raycast:master Jun 14, 2021
@qeude qeude deleted the add/playgrounds branch June 14, 2021 18:33
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.

None yet

3 participants