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

Remove force push #2

Closed
mhkeller opened this issue Jul 25, 2014 · 1 comment
Closed

Remove force push #2

mhkeller opened this issue Jul 25, 2014 · 1 comment

Comments

@mhkeller
Copy link
Contributor

When you swoop deploy it scrubs the deploy trigger from the commit message. This isn't completely necessary because of other security precautions like publish teams and push access. It also causes a git merge on the kestrel server. This second push should be removed. If you really want to scrub your commits, perhaps there is a command that will do that on archive or something like swoop clean that redoes your commits en masse.

@mhkeller
Copy link
Contributor Author

mhkeller commented Aug 8, 2014

Force push has been removed here: 8c08c4c

Added this issue to create possible swoop scrub command but I'm not thrilled about that solution and probably isn't something I'd use: #4

@mhkeller mhkeller closed this as completed Aug 8, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant