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

Move package IDs from UUID to slug #30

Open
bates64 opened this issue Apr 30, 2023 · 0 comments · May be fixed by #41
Open

Move package IDs from UUID to slug #30

bates64 opened this issue Apr 30, 2023 · 0 comments · May be fixed by #41
Milestone
2.0.0

Comments

@bates64
Copy link
Owner

bates64 commented Apr 30, 2023 •
edited
Loading

Why?

  • Makes git branch names understandable
  • Makes asset directory names understandable
  • Having a consistent slug is nice for URLs for package registry when that comes about

Implementation:

  • Migration routine for existing packages
    • All(?) UUIDs are valid slugs, so, if we read an ID that looks like a UUID, warn. See below
  • Emit warning if manifest id is not a slugged version of name? i.e. convert to kebab lower case
    • May want to reserve this check for a future merlon publish command
@github-project-automation github-project-automation bot moved this to Ideas/requests in PM mod tooling Apr 30, 2023
@bates64 bates64 moved this from Ideas/requests to Todo in PM mod tooling Apr 30, 2023
@bates64 bates64 changed the title Consider moving from UUIDs to slugs Move package IDs from UUID to slug Apr 30, 2023
@bates64 bates64 moved this from Todo to In Progress in PM mod tooling May 1, 2023
bates64 added a commit that referenced this issue May 1, 2023
@bates64
use slugs rather than UUIDs for package IDs
40f5f68 
Closes #30
@bates64 bates64 moved this from In Progress to Done in PM mod tooling May 9, 2023
@bates64 bates64 added this to the 2.0.0 milestone May 9, 2023
@bates64 bates64 linked a pull request May 9, 2023 that will close this issue
2.0.0 #41
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
PM mod tooling
Status: Done
Milestone
2.0.0
Development

Successfully merging a pull request may close this issue.

2.0.0 bates64/merlon
1 participant
@bates64