Python version classifiers disagree with CI test matrix

Open doolio opened this issue 2 years ago • 1 comments

https://github.com/romanz/trezor-agent/blob/3c911e99a0394278104564092225d67c75e74b99/agents/trezor/setup.py#L25-L28 https://github.com/romanz/trezor-agent/blob/3c911e99a0394278104564092225d67c75e74b99/agents/onlykey/setup.py#L25-L28 https://github.com/romanz/trezor-agent/blob/3c911e99a0394278104564092225d67c75e74b99/agents/ledger/setup.py#L25-L28 https://github.com/romanz/trezor-agent/blob/3c911e99a0394278104564092225d67c75e74b99/agents/keepkey/setup.py#L25-L28 https://github.com/romanz/trezor-agent/blob/3c911e99a0394278104564092225d67c75e74b99/agents/jade/setup.py#L26-L29 https://github.com/romanz/trezor-agent/blob/3c911e99a0394278104564092225d67c75e74b99/agents/fake/setup.py#L27-L30 https://github.com/romanz/trezor-agent/blob/3c911e99a0394278104564092225d67c75e74b99/setup.py#L44

Considering your CI tests against python3.7 to python3.11 https://github.com/romanz/trezor-agent/blob/3c911e99a0394278104564092225d67c75e74b99/.github/workflows/ci.yml#L11

should the above classifiers be updated to reflect this CI test matrix?

Sep 11 '23 09:09 doolio

https://github.com/romanz/trezor-agent/blob/3c911e99a0394278104564092225d67c75e74b99/libagent/device/ui.py#L126 https://github.com/romanz/trezor-agent/blob/3c911e99a0394278104564092225d67c75e74b99/libagent/util.py#L244-L245 https://github.com/romanz/trezor-agent/blob/master/doc/INSTALL.md#7-installation-troubleshooting also mentions Python2.7.

Could also be looked at again when considering this issue.

Sep 11 '23 13:09 doolio