-
Notifications
You must be signed in to change notification settings - Fork 47
[PW-6793] Add new CI and update tests #167
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
Conversation
Could you please remove Python 2.7 from the README as well if it's deprecated |
@wboereboom I've suggested to deprecate Python 2.7. I think we should communicate that to merchants first and actually drop it on a next major release of the library. |
@michaelpaul in that case let's just state now that it's end of support and it will no longer be maintained. Does that still warrant a major update? |
Co-authored-by: Michael Paul <michael@michaelpaul.com.br>
Co-authored-by: Michael Paul <michael@michaelpaul.com.br>
Co-authored-by: Michael Paul <michael@michaelpaul.com.br>
Co-authored-by: Michael Paul <michael@michaelpaul.com.br>
Co-authored-by: Michael Paul <michael@michaelpaul.com.br>
Yep, if a merchant application is still on Python 2, migrating to 3 could be a lot of work. Merchants don't expect to do such effort with minor/patch upgrades of the library. @jillingk Can you please update the
|
Co-authored-by: Michael Paul <michael@michaelpaul.com.br>
This reverts commit b59c91d.
Description
Created new CI pipeline with Tox to run the unittests on push and pull. Didn't add support for python 2.7 since that will be deprecated.