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

feat: add Version.compare to wasm Version #1080

Merged
merged 4 commits into from
Feb 20, 2025
Merged

feat: add Version.compare to wasm Version #1080

merged 4 commits into from
Feb 20, 2025

Conversation

trim21
Copy link
Contributor

@trim21 trim21 commented Feb 20, 2025

Description

@wolfv
Copy link
Contributor

wolfv commented Feb 20, 2025

Hey @trim21 awesome! I just merged (and squashed) the other PR - do you think you can rebase on main? Thank you for helping us!

@trim21 trim21 marked this pull request as ready for review February 20, 2025 07:56
trim21 and others added 2 commits February 20, 2025 15:58
Co-authored-by: Bas Zalmstra <zalmstra.bas@gmail.com>
Co-authored-by: Bas Zalmstra <zalmstra.bas@gmail.com>
@wolfv
Copy link
Contributor

wolfv commented Feb 20, 2025

@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.

@trim21
Copy link
Contributor Author

trim21 commented Feb 20, 2025

@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

@trim21
Copy link
Contributor Author

trim21 commented Feb 20, 2025

done

@wolfv wolfv merged commit a2fd2d4 into conda:main Feb 20, 2025
3 checks passed
@wolfv
Copy link
Contributor

wolfv commented Feb 20, 2025

Congrats on your first contribution! 🎉

travishathaway pushed a commit to travishathaway/rattler that referenced this pull request Feb 26, 2025
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.

3 participants