Skip to content

Conversation

@dvaldivia
Copy link
Collaborator

No description provided.

Signed-off-by: Daniel Valdivia <18384552+dvaldivia@users.noreply.github.com>
Signed-off-by: Daniel Valdivia <18384552+dvaldivia@users.noreply.github.com>
@cesnietor
Copy link
Collaborator

cesnietor commented Dec 8, 2023

The PR does more than just adding tests, it changes how we handle the requestID.
Is that intentional?

@dvaldivia
Copy link
Collaborator Author

it's intentional @cesnietor if you look at what the NewUUID() function was doing, it was wrapping what uuid.NewString() already does

@dvaldivia dvaldivia merged commit 9db5d1e into minio:master Dec 11, 2023
@dvaldivia dvaldivia deleted the coverate-pkg-utils branch December 11, 2023 17:57
cesnietor pushed a commit to cesnietor/console that referenced this pull request Jan 12, 2024
Signed-off-by: Daniel Valdivia <18384552+dvaldivia@users.noreply.github.com>
cesnietor pushed a commit to cesnietor/console that referenced this pull request Jan 12, 2024
Signed-off-by: Daniel Valdivia <18384552+dvaldivia@users.noreply.github.com>
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.

4 participants