-
Notifications
You must be signed in to change notification settings - Fork 180
Stand-in replacement for the UUID package that doesn't require Python #210
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This pull request introduces a new package called better-uuid that provides UUID generation functionality without requiring Python. It uses the native uuidgen command available on Linux and macOS systems, offering a lightweight alternative to Python-based UUID packages.
Key Changes:
- Adds a new package
better-uuidversion 0.1.0 with two triggers for UUID generation - Provides
:uuidtrigger for standard hyphenated UUIDs and:nuuidfor non-hyphenated UUIDs - Includes package metadata, documentation, and MIT license
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| packages/better-uuid/0.1.0/package.yml | Defines two shell command-based triggers for UUID generation using uuidgen |
| packages/better-uuid/0.1.0/_manifest.yml | Contains package metadata including name, version, author, and tags |
| packages/better-uuid/0.1.0/README.md | Provides usage documentation and platform compatibility notes |
| packages/better-uuid/0.1.0/LICENSE | Standard MIT License with 2026 copyright |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A simple package of two useful triggers with safe shell commands.
Copilot has made a few very minor suggestions, most of which I agree with, in addition to my own comments. None of these are deal-breakers, however!
| # Another UUID generation tool | ||
| This is a stand-in replacement for the `uuid` package that doesn't require Python. To prevent conflicts, you should make sure that `uuid` is not installed before adding `better-uuid`. | ||
|
|
||
| Usage is exactly the same: `:uuid` creates a UUID with dashes, and `:nuuid` creates a UUID without dashes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps include a link to the other package, and update the other's README.md to link to this one?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't know that I was allowed to update other READMEs! I'll do this shortly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any updates need approval, but I see no reason why not.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
@smeech I have a question. Do you know why |
|
I think you have to include the |
Interesting... it seems like I can't install from the |
|
Whoopsies! Renaming the branch closed the PR; new pull request at #211. |
See
README.mdfor more info.