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

Drop support for EOL Python versions, add 3.6, fix failing tests. #2

Merged
merged 1 commit into from
Jan 20, 2018

Conversation

gwax
Copy link
Contributor

@gwax gwax commented Jan 19, 2018

I have some features that I would like to contribute to your package for use downstream. This PR updates the versions of Python and other libraries used in testing and adjusts how tests are performed. This PR does not change package functionality.

Copy link
Contributor

@dahlia dahlia left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution. It would be great if some adjustments are made.

.travis.yml Outdated
- if [[ "$PY_VER" != "3.2" ]]; then
codecov;
fi
- codecov
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you maintain the style as it was as possible (such as no indent for list bullets)? The diff is unnecessarily complex so that I could hardly track the actual changes.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@@ -3,7 +3,7 @@
from sqlalchemy.types import DateTime, TypeDecorator

__all__ = 'UtcDateTime', '__version__'
__version__ = '0.9.0'
__version__ = '0.9.1'
Copy link
Contributor

Choose a reason for hiding this comment

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

If we're going to drop older versions of Python and SQLAlchemy we'd better bump its minor version, i.e., 0.10.0 instead of 0.9.1.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@gwax gwax force-pushed the updated_versions branch from 57460e5 to 79cb05b Compare January 19, 2018 18:05
@codecov-io
Copy link

codecov-io commented Jan 19, 2018

Codecov Report

Merging #2 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master       #2   +/-   ##
=======================================
  Coverage   96.55%   96.55%           
=======================================
  Files           1        1           
  Lines          29       29           
=======================================
  Hits           28       28           
  Misses          1        1
Impacted Files Coverage Δ
sqlalchemy_utc.py 96.55% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8f15ba4...18cdc56. Read the comment docs.

@gwax gwax force-pushed the updated_versions branch 2 times, most recently from 6b6bd6f to 9b8301d Compare January 19, 2018 18:17
@dahlia dahlia merged commit 3d4cf0e into spoqa:master Jan 20, 2018
@dahlia
Copy link
Contributor

dahlia commented Jan 20, 2018

Thanks!

@gwax gwax deleted the updated_versions branch January 20, 2018 18:58
dahlia added a commit that referenced this pull request Jan 25, 2018
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