Skip to content

Conversation

@r24mille
Copy link
Contributor

@r24mille r24mille commented Sep 3, 2017

I would like to upgrade further, but don't think it's prudent until we get
unit tests and integration tests sorted in #130. Then we could probably
bump the version up further.

For now, I think upgrading the dependency from 3.6.1 to 3.6.4 is fairly
safe (unit tests pass) and the version change bring in stabilization fixes
only. See http://lxml.de/3.6/changes-3.6.4.html for details.

 I would like to upgrade further, but don't think it's prudent until we get
 unit tests and integration tests sorted in WattTime#130. Then we could probably
 bump the version up further.

 For now, I think upgrading the dependency from 3.6.1 to 3.6.4 is fairly
 safe (unit tests pass) and the version change bring in stabilization fixes
 only. See http://lxml.de/3.6/changes-3.6.4.html for details.
@ajdonnison ajdonnison merged commit 4d6a395 into WattTime:master Sep 4, 2017
@r24mille r24mille deleted the 108_lxml_dependency_upgrade branch September 4, 2017 12:07
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.

2 participants