Skip to content
This repository was archived by the owner on May 6, 2020. It is now read-only.

Conversation

@bacongobbler
Copy link
Member

closes #28

Copy link
Member

Choose a reason for hiding this comment

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

Why was this removed? Without that flag you get the annoying upgrade message all the time.

Copy link
Member Author

Choose a reason for hiding this comment

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

just sticking to being consistent with lines 31 and 32 in .travis.yml. I can revert this if you'd prefer :)

Copy link
Member

Choose a reason for hiding this comment

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

Oh I see. Now I'm torn between improving consistency and reducing pointless log messages...

Meh, this is trivial either way, LGTM.

@mboersma mboersma added the LGTM1 label Dec 4, 2015
bacongobbler pushed a commit that referenced this pull request Dec 7, 2015
feat(travis): hook up docs to travis
@bacongobbler bacongobbler merged commit d002f3e into deis:master Dec 7, 2015
@bacongobbler bacongobbler deleted the 28-docs-on-travis branch December 7, 2015 18:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

hook up docs to travis

3 participants