Skip to content

Conversation

@paulftw
Copy link

@paulftw paulftw commented Jul 22, 2011

Hi David,

This is the setup.py I've talked to you earlier today.
Please insert your email address in it (or drop that line completely).

Let me know what you think.

I've also added another demo, to show how to quickly download and install the latest version of your library.
If you wish to incorporate it in your project, don't forget to edit the URL in buildout.cfg to point to your repository.

@paulftw
Copy link
Author

paulftw commented Jul 22, 2011

I've added support for lifetime=0 from a fork by rimmington.

@toqueteos
Copy link

@dound Is this pull gonna get pushed to master?

@dound
Copy link
Owner

dound commented Sep 6, 2011

Just reviewed the diff. I think the change to the library itself (gaesessions/init.py) probably wasn't intended to be part of the pull request:

https://github.com/dound/gae-sessions/pull/28/files#diff-7

@paulftw
Copy link
Author

paulftw commented Sep 8, 2011

Reverted the change to gaesessions, you are right, it should be reviewed separately.

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