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.

Sign up for GitHub

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

Jump to bottom

Impl HashEqLike<T> for &T #671

Merged
merged 1 commit into from
Feb 5, 2025
Merged

Impl HashEqLike<T> for &T #671

merged 1 commit into from
Feb 5, 2025

Conversation

Veykril
Copy link
Member

@Veykril Veykril commented Feb 5, 2025 •
edited
Loading

Fixes #670

@netlify Netlify
Copy link

netlify bot commented Feb 5, 2025 •
edited
Loading

✅ Deploy Preview for salsa-rs canceled.

Name Link
🔨 Latest commit 73a353c
🔍 Latest deploy log https://app.netlify.com/sites/salsa-rs/deploys/67a38b1dc72f7f0008b98890

@Veykril
Copy link
Member Author

Veykril commented Feb 5, 2025

Merely forgot to duplicate that one when I split the traits
r? @ibraheemdev

@codspeed-hq CodSpeed HQ
Copy link

codspeed-hq bot commented Feb 5, 2025

CodSpeed Performance Report

Merging #671 will not alter performance

Comparing Veykril:veykril/push-vowtzoxmkqtq (73a353c) with master (ae416f1)

Summary

✅ 9 untouched benchmarks

ibraheemdev
ibraheemdev approved these changes Feb 5, 2025
View reviewed changes
Copy link
Contributor

@ibraheemdev ibraheemdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect, thanks.

@Veykril Veykril added this pull request to the merge queue Feb 5, 2025
Merged via the queue into salsa-rs:master with commit acd842a Feb 5, 2025
10 checks passed
@Veykril Veykril deleted the veykril/push-vowtzoxmkqtq branch February 5, 2025 17:38
@ibraheemdev ibraheemdev mentioned this pull request Feb 7, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ibraheemdev ibraheemdev ibraheemdev approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Missing HashEqLike impl for &T
2 participants
@Veykril @ibraheemdev