pcpp icon indicating copy to clipboard operation
pcpp copied to clipboard

Basic tox with pyenv setup for testing

Open assarbad opened this issue 2 years ago • 5 comments

  • Amended ignore file
  • Added tox.ini to test an array of Python versions
  • .python-version file (used by pyenv) to enable concurrent availability of Python installations

Hope you find this suitable. If not I won't take offense. But I wanted to be able to test across a number of Python versions beyond just 3.7 and 2.7.

assarbad avatar Nov 13 '22 22:11 assarbad

Thanks for the PR. Doesn't tox also support windows?

ned14 avatar Nov 14 '22 15:11 ned14

Thanks for the PR. Doesn't tox also support windows?

It does, but before adding it, I want to get Linux straightened out. And when that all works, I can deal with Windows, I suppose. One item at a time, though. (And I can't even ever test macOS.)

assarbad avatar Nov 14 '22 21:11 assarbad

Will you promise to me that if I merge this now that you will contribute tox support for windows?

ned14 avatar Nov 16 '22 14:11 ned14

Will you promise to me that if I merge this now that you will contribute tox support for windows?

Frankly. Nope. And the reason isn't because I am lazy or unwilling to do it (in fact chances are you will see this from me). But I am unable to commit this and give you a reasonable time frame (and by reasonable I mean anything I would deem reasonable for my own projects).

If you don't want to merge it, that's fine with me. I'll simply try to maintain a branch with this patch rebased on latest master from you (and another branch combining my own as-of-yet unmerged patches). This way I can make use of it and drive the effort of increasing test coverage all the while not committing to a fixed deadline, which would seem dishonest to you if I don't make the deadline all the while making me feel terrible for not having made it 😉

Git for Windows uses a similar approach to stay "afloat" with the Windows-specific patches against the upstream master branch.

assarbad avatar Nov 16 '22 19:11 assarbad

Thanks for the honesty! I'm going to reopen this PR however, as roughly once per year I try to close all open bug count, merge all open PRs and do a release. If when that time comes this is still open I'll implement the windows support and merge it.

ned14 avatar Nov 18 '22 09:11 ned14