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

Test git #2

Merged
merged 4 commits into from
Jun 7, 2012
Merged

Test git #2

merged 4 commits into from
Jun 7, 2012

Conversation

sosedoff
Copy link
Contributor

@sosedoff sosedoff commented Jun 6, 2012

Hey, i did a bunch of small improvements.

Also, i encountered the problem if you try to run deploy outside of git tree. Im using mina to run automated deployments, so on my side there are no git repositories or project files, basically running it in headless mode. My change fixes that issue.

Test is included.

@rstacruz
Copy link
Member

rstacruz commented Jun 7, 2012

Thank you for cleaning up our mess! :)

@rstacruz
Copy link
Member

rstacruz commented Jun 7, 2012

...and thank you for putting tests in there as well. @mikong this is clear with me. :-)

mikong added a commit that referenced this pull request Jun 7, 2012
Check Git if empty, gitignore pkg, and fix deploy script template.
@mikong mikong merged commit df9ae68 into mina-deploy:master Jun 7, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants