Call config.addini to eliminate warnings in downstream repos #26
Conversation
tbekolay
force-pushed
the
fix-addini
branch
2 times, most recently
from
d182833 to
00d211d
Compare
|
These changes (e.g. using f-strings) will mean that we no longer support Python 3.5. I'm fine with this (I noticed you also changed the tests to no longer test 3.5, so I'm guessing you are too), but we should update both the min python version and the classifiers in nengobones.yml accordingly. (Also, assuming the tests pass on 3.8/3.9, we can add those classifiers.) |
|
Yes, Python 3.5 is past its end of life so we have no plans to support it across any of our projects. The min Python version (in NengoBones) defaults to 3.6, but yes we should definitely change the classifiers. Made that change, and updated appveyor to hopefully pass this time ... |
tbekolay
force-pushed
the
fix-addini
branch
2 times, most recently
from
6df5bc5 to
cf80513
Compare
And update .nengobones.yml to reflect recent changes.
This eliminates warnings that were being raised in packages using the config options exposed by pytest-plt. This commit also fixes style issues raised by more recent version of NengoBones, Pylint, flake8, etc.
|
CI is all passing now; coverage isn't being collected correctly, but I think we should not worry about that for now. |
Warnings were being thrown in NengoCore because pytest-plt didn't call
config.addinifor configuration options.This PR also updates to the lastest version of NengoBones.