Skip to content
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

Follow django Python support map, add test for stable django 1.10 and… #37

Merged
merged 1 commit into from Sep 11, 2017

Conversation

hsmett
Copy link
Contributor

@hsmett hsmett commented Aug 25, 2017

… 1.11

@carltongibson
Copy link
Contributor

Hi @hsmett

Thanks for this.

The CI failed for Django 1.8 and Python 3.2. “Interpretor Not Found”

At this point I would be happy to just drop that. (Although if you’re happy to battle Travis, you are welcome)

@carltongibson carltongibson self-assigned this Sep 4, 2017
@hsmett
Copy link
Contributor Author

hsmett commented Sep 11, 2017

Hello @carltongibson, sorry it took me so long to answer!
I didn't noticed the Travis configuration file, I'm going to update it based on the official Django Python support:
https://docs.djangoproject.com/en/1.11/faq/install/
as I did for tox.
And maybe add Python 3.6 tests if it doesn't take me too much time :)

@hsmett hsmett force-pushed the develop branch 5 times, most recently from 4b39f4f to e7183bf Compare September 11, 2017 11:48
@hsmett
Copy link
Contributor Author

hsmett commented Sep 11, 2017

Should be good :)

Copy link
Contributor

@carltongibson carltongibson left a comment

Choose a reason for hiding this comment

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

That’s lovely. Excellent job. Thank you!

@carltongibson carltongibson merged commit 7b02063 into django-compressor:develop Sep 11, 2017
@hsmett
Copy link
Contributor Author

hsmett commented Sep 14, 2017

My pleasure! Do you plan to make a new release soon?

@carltongibson
Copy link
Contributor

Not really. There hasn't been a code change since the last release. There's nothing to release...

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.

None yet

2 participants