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
add uint64 and uint to pointer pkg #253
add uint64 and uint to pointer pkg #253
Conversation
|
Welcome @kschoche! |
I'd sort of expect all these things to now be defined by golang since I'm assuming they can be implemented with generics? |
Hi @apelisse, thanks for the reply! The use case I'm trying to address is to provide coverage in the pointer package for Uint64 as we are trying to use it with some CRD code that uses it for things like versions and timestamps. I do understand that an obvious+easy answer is for users to BYO functionality, which is what we already do, and so this PR is more of a "nice to have" vs a necessity. Thoughts? |
I guess I don't see any problem with this PR being merged in the mean time. Where do you need this code? Or is it strictly hypothetical? |
Cool. |
Hi @apelisse - sorry for the ping here, but I just wanted to double-check if there were any additional changes you might see needed for this PR while it's fresh in my mind. Cheers. |
Sorry, I missed your reply. /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: apelisse, kschoche The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind feature
What this PR does / why we need it:
This PR adds support for interacting with 32 and 64bit unsigned ints in the
pointer
package.We wanted to standardize on the pointer package but found that it was lacking unsigned int support which is commonly used in k8s and applications interacting with k8s.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
I'm not 100% sure if it makes sense to add the back-compatibility pieces which I copied from the existing
Int
andInt64
implementations. If not I can back them out and just leave the new pieces.Release note: