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

Add static Runnables #8009

Merged
merged 106 commits into from
Feb 19, 2024
Merged

Add static Runnables #8009

merged 106 commits into from
Feb 19, 2024

Conversation

osiewicz
Copy link
Contributor

@osiewicz osiewicz commented Feb 19, 2024

Partially addresses #7108 .

This PR includes just the static runnables part. We went with not having a dedicated panel for runnables.
This is just a 1st PR out of N, as we want to start exploring the dynamic runnables front. Still, all that work is going to happen once this gets merged.
TODO:

  • docs
  • Schema support in runnables definition file.
  • Use serde_json_lenient parser.

Release Notes:

  • Added Runnables support to Zed. Runnables are defined in runnables.json file (accessible via zed: open runnables action) and they can be spawned with runnables: spawn action.

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 19, 2024
@osiewicz osiewicz changed the title Add Runnables Add static Runnables Feb 19, 2024
SomeoneToIgnore and others added 3 commits February 19, 2024 12:45
Try to use the cwd from runnable definition first, and if it's absent,
fallback to the current project's cwd detection and even if it fails, move on with runnable's execution.
@SomeoneToIgnore
Copy link
Contributor

cc @mikayla-maki as you were interested in the terminal part.
Will merge this optimistically, but please send the feedback if you find something interesting.

@SomeoneToIgnore SomeoneToIgnore mentioned this pull request Feb 19, 2024
1 task
@SomeoneToIgnore SomeoneToIgnore merged commit f17d0b5 into main Feb 19, 2024
7 checks passed
@SomeoneToIgnore SomeoneToIgnore deleted the runnables branch February 19, 2024 16:41
@SomeoneToIgnore SomeoneToIgnore mentioned this pull request Feb 21, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed The user has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants