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

A tiny extra piece of configurability will make Octopress preview work on ... #780

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
5 participants
@maxlinc

maxlinc commented Oct 4, 2012

...c9.io out of the box.

@janjongboom

This comment has been minimized.

Show comment
Hide comment
@janjongboom

janjongboom Oct 7, 2012

+1 for this pull request. The environment variables are used on more envs than just Cloud9.

janjongboom commented Oct 7, 2012

+1 for this pull request. The environment variables are used on more envs than just Cloud9.

@systole

This comment has been minimized.

Show comment
Hide comment
@systole

systole Nov 14, 2012

+1 for C9.io

systole commented Nov 14, 2012

+1 for C9.io

@ralgh

This comment has been minimized.

Show comment
Hide comment
@ralgh

ralgh Nov 28, 2012

+1, Works great

ralgh commented Nov 28, 2012

+1, Works great

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Dec 17, 2012

Collaborator

Hm, let me just make sure this doesn't break rake preview.

Collaborator

parkr commented Dec 17, 2012

Hm, let me just make sure this doesn't break rake preview.

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Dec 17, 2012

Collaborator

Works for me. I'll merge the changes into 2.1.

Collaborator

parkr commented Dec 17, 2012

Works for me. I'll merge the changes into 2.1.

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Dec 17, 2012

Collaborator

Merged in 329d2c5.

Collaborator

parkr commented Dec 17, 2012

Merged in 329d2c5.

@parkr parkr closed this Dec 17, 2012

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Dec 17, 2012

Collaborator

Thinking about this more, this can really f up some things on production servers. Going to still use ENV vars, but favour OCTOPRESS_-prefixed ones.

Collaborator

parkr commented Dec 17, 2012

Thinking about this more, this can really f up some things on production servers. Going to still use ENV vars, but favour OCTOPRESS_-prefixed ones.

parkr added a commit that referenced this pull request Dec 17, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment