-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
refac: deprecate vitess/go/maps2
for golang.org/x/exp/maps
#14960
Conversation
Review ChecklistHello reviewers! 👋 Please follow this checklist when reviewing this Pull Request. General
Tests
Documentation
New flags
If a workflow is added or modified:
Backward compatibility
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this is the right change here. What we want is to remove maps2
since these are functions that Go provides these days in golang.org/x/exp/maps
.
@EshaanAgg Is this something you are interested in tackling? Otherwise I can clean it up as well. |
@dbussink Thanks for the review! I have cleaned up as you instructed. Would be renaming the PR to reflect the same. |
vitess/go/maps2
vitess/go/maps2
for golang.org/x/exp/maps
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice cleanup!
The code coverage workflow seems to fail, but I can't seem to figure out the reason for the same (can't see any changes in this PR that should affect those tests). Can someone help me debug that? |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #14960 +/- ##
==========================================
- Coverage 47.26% 47.25% -0.01%
==========================================
Files 1138 1138
Lines 238842 238842
==========================================
- Hits 112880 112864 -16
- Misses 117368 117377 +9
- Partials 8594 8601 +7 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Eshaan Aggarwal <[email protected]>
Signed-off-by: Eshaan Aggarwal <[email protected]>
Signed-off-by: Eshaan Aggarwal <[email protected]>
Signed-off-by: Eshaan Aggarwal <[email protected]>
Signed-off-by: Eshaan Aggarwal <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution.
Description
Remove unneeded
maps2
package sincegolang.org/x/exp/maps
provides these as well.Related Issue(s)
Fixes part of #14931
Checklist
Deployment Notes