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

Scaffold commands shouldn't overwrite existing files #17

Closed
buildmaster opened this issue Dec 17, 2018 · 1 comment · Fixed by #88
Closed

Scaffold commands shouldn't overwrite existing files #17

buildmaster opened this issue Dec 17, 2018 · 1 comment · Fixed by #88
Labels
bug Something isn't working cli Relates to `zapier-platform-cli`

Comments

@buildmaster
Copy link
Contributor

What would you like to change?

Currently Scaffold commands will rewrite the contents of the file created by the scaffold command if it already exists. As such work can be lost by accidentally running the scaffold command on an existing resource/trigger/action.

We should error out of the scaffold command and warn the user that the file already exists. Or provide them an option to force an overwrite.

@zapzap
Copy link

zapzap commented Apr 25, 2019

This issue has been copied into our private issue (as PDE-865) tracker. Thanks for the report! We'll update this as we learn more.

@xavdid xavdid transferred this issue from zapier/zapier-platform-cli Jun 25, 2019
@xavdid xavdid added bug Something isn't working cli Relates to `zapier-platform-cli` labels Jun 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cli Relates to `zapier-platform-cli`
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants