Skip to content
This repository has been archived by the owner on Feb 18, 2019. It is now read-only.

Add tox.ini #6

Merged
merged 1 commit into from
Aug 1, 2012
Merged

Add tox.ini #6

merged 1 commit into from
Aug 1, 2012

Conversation

carljm
Copy link
Contributor

@carljm carljm commented Jul 20, 2012

We should add a tox.ini to automatically test dzclient on multiple Python versions (and consider supporting 2.4 so Talos can use it).

@carljm
Copy link
Contributor Author

carljm commented Jul 20, 2012

This makes it easy to make sure we don't break compatibility with Python 2.5+.

I took a look at also adding 2.4 compat, but there are compatibility issues in oauth2 that I don't want to deal with here. Talos will be bundling copies of this stuff anyway, it'll make more sense to deal with that stuff there, since that's the only place I expect to be using this under 2.4 anyway.

@malini
Copy link
Contributor

malini commented Aug 1, 2012

/me kicks github

I did not see this pull request nor get email notifications for any of your comments, so sorry for this CRAZY delay. I'm getting a bugzilla component for Datazilla, so we can get all patch reviews up there. It's much easier that way since notifications are much louder, and people can subscribe to the new bugzilla component.

malini added a commit that referenced this pull request Aug 1, 2012
@malini malini merged commit f7f59aa into mozilla:master Aug 1, 2012
@carljm
Copy link
Contributor Author

carljm commented Aug 6, 2012

No worries, I wasn't waiting on this - thanks for merging!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants