Skip to content

Deployment Improvements - #119

Open
FadyKalo-zz wants to merge 2 commits into
masterfrom
deploy_improvements
Open

Deployment Improvements#119
FadyKalo-zz wants to merge 2 commits into
masterfrom
deploy_improvements

Conversation

@FadyKalo-zz

Copy link
Copy Markdown
Collaborator
  • login page uses bootsrap, shall we need bower static? or shall we use seevcam.css in login page?
  • Versioning
  • make file
  • Refactoring client folder
  • automatically branch detection
  • fix ngixn reload
  • fix gunicon crashes
  • add gunincorn logs

@fadykalo I will use this branch to do my deployment improvements. \do you have anything left to do here?

@FadyKalo-zz

Copy link
Copy Markdown
Collaborator Author

Using local then the downtime is optimized, really minimal.
I don' like having multiple tasks local and remote, i will refactor to use for example: minify:local: or minify:remote so that we can keep tasks tidy.

@giuse88

giuse88 commented Jan 25, 2015

Copy link
Copy Markdown
Owner

nice :)

@giuse88

giuse88 commented Jan 25, 2015

Copy link
Copy Markdown
Owner

The login page use Bootstrap. However Boostrap is included in seevcam.min.js and seevcam.min.css therefore you can include those files instead of Boostrap.

Comment thread Gruntfile.js

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why do you need to specify the branch here?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is the config file for shipit where you tell it the branch to deploy, since we haven't merged in master I used the deploy-improvements to deploy my updates.

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

but the script should be able to detect which branch you are working on and deploy it.

@FadyKalo-zz FadyKalo-zz self-assigned this Jan 27, 2015
@giuse88
giuse88 force-pushed the master branch 2 times, most recently from 588bc14 to 60eac9c Compare March 3, 2015 08:02
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