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

fix: filemanager uuids #161

Merged
merged 3 commits into from
Mar 20, 2024
Merged

fix: filemanager uuids #161

merged 3 commits into from
Mar 20, 2024

Conversation

mmalenic
Copy link
Member

Fixes #159, fixes #160

Changes

  • Remove unnecessary Cargo.lock inside the filemanager crate because the top-level workspace has a Cargo.lock.
  • Add UuidGenerator struct to standardise uuids in filemanager using v7.
  • Update uuid dependency.

@mmalenic mmalenic requested a review from brainstorm March 18, 2024 05:45
@mmalenic mmalenic self-assigned this Mar 18, 2024
@mmalenic mmalenic added filemanager an issue relating to the filemanager fix labels Mar 18, 2024
@mmalenic mmalenic merged commit 169f776 into main Mar 20, 2024
2 checks passed
@mmalenic mmalenic deleted the fix/filemanager-uuids branch March 20, 2024 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brainstorm brainstorm brainstorm approved these changes

Assignees

@mmalenic mmalenic

Labels
filemanager an issue relating to the filemanager fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

filemanager: upgrade uuid v4 to v7 filemanager: creating a vec of uuids clones the first uuid
2 participants
@mmalenic @brainstorm