-
Notifications
You must be signed in to change notification settings - Fork 106
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
More tests, more docs #12
Comments
Hi all, If I was to get involved with the project - are there any suggestions for where adding unit tests would be most valuable? |
@cjdsellers integration tests against the reference zeromq implementation would be very valuable - specifically making sure that the library is compliant with the spec. |
Ok understood. I've working with ZMQ before in both C# and Python and integrated services between those languages. I'll familiarize myself with the spec some more. |
What I'm actually doing when working on some new functionality:
Maybe we could do the same for integrations tests so for example test Rust implementation against some python code (cause it requires only a few lines of code). Need to think about how this can be run locally and on CI. Possibly in a docker container? |
@Alexei-Kornienko its probably fine to just rely on the |
@TheButlah yeah but still it's a bit harder to setup and configure.. maybe it's faster however for me Python is much easier to bootstrap. |
No description provided.
The text was updated successfully, but these errors were encountered: