Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Run tests on BSDs, too. #86

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

trombonehero
Copy link
Contributor

Now that FreeBSD support has been added to readchar/key.py, these tests run just fine.

Now that FreeBSD support has been added to readchar/key.py, these tests
run just fine.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 98.235% when pulling 4636e0c on trombonehero:bsd-tests into 01b4b0a on magmax:master.

@Cube707
Copy link
Collaborator

Cube707 commented Jul 27, 2022

Thank you for this PR. I also thought about this, but if this gets incorporated we would also have to extend the CI test-workflow. And than comes the question, why only BSD? Why not MacOS also? And others?

This gets more complicated than I am willing to handle and also would require somebody to handle isses that come up from this. This is also why the testfolder is still named Linux and not Posix like the sourcefiles. Tests are only provided / supported for Linux and Windows and more is not actively maintained.

Everyone is of course free to remove the guard-staments and run the tests on this own risks.

@Cube707 Cube707 added wontfix this won't be worked on, feel free to contribute out of scope this is currently not managable (but may be in the future) and removed wontfix this won't be worked on, feel free to contribute labels Jul 28, 2022
Repository owner locked as resolved and limited conversation to collaborators Nov 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
out of scope this is currently not managable (but may be in the future)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants