-
Notifications
You must be signed in to change notification settings - Fork 70
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
feat: add Version.compare to wasm Version #1080
Conversation
Hey @trim21 awesome! I just merged (and squashed) the other PR - do you think you can rebase on main? Thank you for helping us! |
Co-authored-by: Bas Zalmstra <zalmstra.bas@gmail.com>
Co-authored-by: Bas Zalmstra <zalmstra.bas@gmail.com>
@trim21 you marked @baszalmstra comments as resolved but didn't do the changes yet. I am happy to merge once you did apply the suggestions. |
Oh, sorry, my mistake. I want to trigger the required ci jobs so did a force push, but with a old commit |
done |
Congrats on your first contribution! 🎉 |
Description