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.

Sign up for GitHub

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

Jump to bottom

APP_PATH #2

Closed

rnixx opened this issue May 23, 2011 · 1 comment

Member

rnixx commented May 23, 2011

cone.app.init

XXX: is this needed anywhere? If not, let's remove it.

APP_PATH = os.environ.get('APP_PATH', '')

APP_PATH is currently used to determine the directory containing file based application configuration. So, yes, it's needed, but the
name is probably confusing, or at least not very significant.

We might rename this. Suggestions?

Contributor

attilaolah commented May 24, 2011

Hi,

On Mon, May 23, 2011 at 09:17, rnixx
reply@reply.github.com
wrote:

We might rename this. Suggestions?

I actually have no problem with the name, as long as it is used with
.get('APP_PATH') and not directly with ['APP_PATH']. I just thought
that you don't need in anymore, that's why I put the comment there.

rnixx closed this as completed

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