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

'stack upload' should require confirmation before uploading #439

Closed
borsboom opened this issue Jun 27, 2015 · 3 comments
Closed

'stack upload' should require confirmation before uploading #439

borsboom opened this issue Jun 27, 2015 · 3 comments

Comments

@borsboom
Copy link
Contributor

@borsboom borsboom commented Jun 27, 2015

I was originally going to title this "'stack upload' should be harder to use."

Uploading a package to hackage is irrevocable and it's a too easy now to accidentally upload without intending to do so (which I did in the last stack release, so now there's a weird version of stack on Hackage that can't be deleted). stack upload could default to just telling you what it would do plus a message saying something like "use stack upload -y to actually upload". Alternatively, it could prompt the user interactively, and also provide a -y option to have it skip the prompt.

@snoyberg snoyberg added this to the 0.2.0.0 milestone Jun 28, 2015
@snoyberg
Copy link
Contributor

@snoyberg snoyberg commented Jun 28, 2015

I hadn't thought of going in this direction to be honest. What I'd thought of was simply removing the ability of stack upload without arguments to do an upload.

@borsboom
Copy link
Contributor Author

@borsboom borsboom commented Jun 28, 2015

Ah, so you'd have to specify the directory/tarball. Yeah, that'd work too. Just something more than stack upload on its own.

snoyberg added a commit that referenced this issue Jun 29, 2015
@snoyberg
Copy link
Contributor

@snoyberg snoyberg commented Jun 29, 2015

@manny-fp Confirmed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.