Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

Add wrangler delete for scripts #2086

Closed
wants to merge 11 commits into from
Closed

Add wrangler delete for scripts #2086

wants to merge 11 commits into from

Conversation

ocsfrank
Copy link
Contributor

@ocsfrank ocsfrank commented Sep 24, 2021

This PR adds a command wrangler delete that allows to delete a Workers script.

wrangler delete will delete a script based on the script name and account ID found in the wrangler.toml file.

Without any flags, the command will ask for confirmation steps when deleting the script, and any bound Durable Objects.
It's also possible to pass a flag --force to skip all the confirmation steps. WARNING: --force will also delete the Durable Objects bound to a script.

Should solve #210.

src/cli/delete.rs Outdated Show resolved Hide resolved
src/cli/mod.rs Outdated Show resolved Hide resolved
src/cli/mod.rs Outdated Show resolved Hide resolved
src/commands/delete/mod.rs Outdated Show resolved Hide resolved
src/cli/mod.rs Outdated Show resolved Hide resolved
src/commands/delete/mod.rs Outdated Show resolved Hide resolved
@ocsfrank ocsfrank marked this pull request as ready for review September 28, 2021 21:09
@ocsfrank ocsfrank requested a review from a team as a code owner September 28, 2021 21:09
@ocsfrank
Copy link
Contributor Author

Changed to read all project info from wrangler.toml and added a PR to cloudflare-rs (cloudflare/cloudflare-rs#166) for adding the API endpoints.

@JacobMGEvans
Copy link
Contributor

Thanks for creating this PR! Wrangler v1 is now deprecated and support is only being provided for critical updates or security concerns. As such, we are closing this PR.

New versions of Wrangler are maintained in the workers-sdk repo. If you are running into a similar issue with wrangler v2, please report it or create a new PR in the workers-sdk repo. For more info about wrangler v1 deprecation, please check out our blog post.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants