-
Notifications
You must be signed in to change notification settings - Fork 751
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 approval tests to the public API surface #479
Comments
Totally up for helping this along. Is targeting .Net 45/462 to start with, and expand to other platforms if we can make it work a sensible strategy? |
Yup let's start with net471 to Rx/Ix and add other platforms as as seperate PRs as needed as we learn more by adding those platforms as integration tests to ReactiveUI. |
I'm currently digging into getting this added for the projects under the Rx.NET folder. I'm planning to carry on into the other folders (Ix.NET etc) once I get a PR in for the Rx.NET stuff but I'm happy to collaborate. |
It's yours for next 72 hours. Good luck. Olly can help if you need a partner. Just ask. |
Fixed by #482 |
We need to make it more accessible and safer for folks to contribute to this project. I would love to do a performance pass but not before basics such as increasing test coverage happen.
So this is a help wanted? Can you help out? Will you.
Deliverables
The text was updated successfully, but these errors were encountered: