Pluggy should be in install_requires; pytest shouldn’t

[alexshpilkin]

Since #8, there has been a dependency on pluggy which is not listed in install_requires / requirements.txt.

On the other hand, as best as I can tell, pytest is only required to run tests (which is reasonable), yet is present in install_requires instead of only in tests_require, thus requiring every riscv_isac user to have it on their computer (which is not). It would be better to move it to the latter.

[pawks]

@alexshpilkin Thanks for filing the issue. Would you be willing to raise a PR for the same?

[alexshpilkin]

@pawks I’m fine doing that, sure. Do you want to make a separate test_requirements.txt or just special-case pytest for now? (I could even imagine comment-separated sections in the current requirements.txt file, but that sounds more fancy that this tiny problem is worth, honestly.)

[pawks]

Its better to have a separate test_requirements.txt than try to separate the requirements.txt into sections.