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

Add some rudimentary testing and run it in GitHub CI #34

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

dabrahams
Copy link
Contributor

The tests should exercise the error capture functionality; they currently do not.

Windows support is coming, but expect it to be a much bigger change because of the many workarounds required.

@dabrahams
Copy link
Contributor Author

…and Windows support is implemented now. That PR depends on this one, so it's still on my fork, but it will be re-targeted at this repository if this PR is merged.

@dabrahams
Copy link
Contributor Author

@roop Realistically speaking, is there any chance of my improvements being merged? I don't want to waste time submitting PRs here if you're abandoning this, or if you just don't approve of what I'm doing.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant