Skip to content

Conversation

@Mr-Leshiy
Copy link
Contributor

Description

Make a uuid::Uuid dependency import pub, so it could be possible to reuse uuid::Uuid on the clients code without explicit adding a uuid dependency.

@Mr-Leshiy Mr-Leshiy self-assigned this Feb 2, 2025
@Mr-Leshiy Mr-Leshiy added the review me PR is ready for review label Feb 2, 2025
@Mr-Leshiy Mr-Leshiy requested a review from bkioshn February 2, 2025 16:40
@Mr-Leshiy Mr-Leshiy changed the title feat(rust/catalyst-types) uuid::Uuid public import feat(rust/catalyst-types): uuid::Uuid public import Feb 2, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Feb 2, 2025

Test Report | ${\color{lightgreen}Pass: 273/273}$ | ${\color{red}Fail: 0/273}$ |

@stanislav-tkach
Copy link
Member

I have a question. Do we strive to only use specific UUID types such as v4 and v7? In this case the change simplifies usage of the "raw" UUID type.

@Mr-Leshiy Mr-Leshiy merged commit 13b40c7 into main Feb 3, 2025
22 of 23 checks passed
@Mr-Leshiy Mr-Leshiy deleted the feat/cat-types-uuid branch February 3, 2025 08:45
@Mr-Leshiy
Copy link
Contributor Author

@stanislav-tkach

Do we strive to only use specific UUID types such as v4 and v7?

As far as I know yes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review me PR is ready for review

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants