setup: disable test command#4107
Conversation
086d14a to
dcfa72d
Compare
|
|
I think the message is fine without any links. I already mentioned earlier that I want to move most of the metadata defined imperatively in setup.py into the declarative setup.cfg format and clean up everything. The package metadata is a bit messy right now and hasn't been touched in ages. There were a lot of changes in the past years in regards to python packaging standards, and we should stay up2date. It's a bit difficult though following and understanding all the changes here while having the supported python versions, dependencies and downstream system packages in mind. After looking at the setuptools keywords docs, I found the following about the
Should we remove it in this PR, or should we do this when cleaning up the package metadata? Removing it instead of adding the sys.exit call doesn't have any effect though, despite what the documentation is saying. Either way, it should be removed at some point. |
Doing it in the clean up PR makes more sense. I think this is fine so I'm going to merge it in now. |
See #4102
Let's keep that issue thread open though, so that package maintainers who are running tests incorrectly in their package configs can see it.
"test" in sys.argv?