Skip to content

Conversation

@PPPDUD
Copy link
Contributor

@PPPDUD PPPDUD commented Jan 9, 2026

See #210.

@PPPDUD
Copy link
Contributor Author

PPPDUD commented Jan 9, 2026

@smeech Can you show me an example of how I can make it use WSL on Windows and ignore it elsewhere? I'm sure that this is obvious, but I'm new to Espanso development.

Copy link
Contributor

Copilot AI left a 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 PR introduces a new uuid-nix package as a Bash-based alternative to the existing Python-based uuid package, and updates the original uuid package's README to reference this new option.

  • Adds uuid-nix package (version 0.1.0) that uses uuidgen command instead of Python
  • Updates uuid package README to inform users about the new alternative
  • Implements :uuid and :nuuid triggers using shell commands

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
packages/uuid/1.0.0/README.md Adds documentation about the new uuid-nix package as an alternative
packages/uuid-nix/0.1.0/package.yml Implements UUID generation triggers using uuidgen and sed commands
packages/uuid-nix/0.1.0/_manifest.yml Defines package metadata, tags, and authorship information
packages/uuid-nix/0.1.0/README.md Provides usage documentation and platform compatibility notes
packages/uuid-nix/0.1.0/LICENSE Includes MIT license for the package

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@PPPDUD
Copy link
Contributor Author

PPPDUD commented Jan 9, 2026

Hold on! The uuid package doesn't seem to match the case of my trigger; :uuid and :UUID do the same thing. Is this intended behavior?

@smeech
Copy link
Collaborator

smeech commented Jan 9, 2026

Can you show me an example of how I can make it use WSL on Windows and ignore it elsewhere?

Just add shell:wsl to each trigger, at the level of cmd:. Non-WSL systems ignore it.

Copy link
Collaborator

@smeech smeech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you.

A few minor changes, but we're nearly there.

@PPPDUD
Copy link
Contributor Author

PPPDUD commented Jan 9, 2026

@smeech I don't think that word: true is the correct approach here; the original UUID extension doesn't use it; perhaps you meant propagate_case?

@PPPDUD
Copy link
Contributor Author

PPPDUD commented Jan 9, 2026

@smeech Offtopic, but I'd just like to say that you have been so nice to me! Thank you for being an excellent person and responding to my questions so quickly! You're the best.

@smeech
Copy link
Collaborator

smeech commented Jan 9, 2026

@smeech Offtopic, but I'd just like to say that you have been so nice to me! Thank you for being an excellent person and responding to my questions so quickly! You're the best.

You're welcome! Thanks.

@smeech
Copy link
Collaborator

smeech commented Jan 10, 2026

I've added the link to your package to the uuid/1.0.0/README.md file (to resolve my second suggestion in #211 (comment)) and inserted a couple of backticks into yours, to save you the trouble.

If you're happy with the package now, I'll go ahead and merge.

@PPPDUD
Copy link
Contributor Author

PPPDUD commented Jan 10, 2026

I've added the link to your package to the uuid/1.0.0/README.md file (to resolve my second suggestion in #211 (comment)) and inserted a couple of backticks into yours, to save you the trouble.

If you're happy with the package now, I'll go ahead and merge.

I think that it's about ready to merge. Thanks so much for helping out with this!

@smeech smeech merged commit 6de6411 into espanso:main Jan 10, 2026
1 check passed
@PPPDUD PPPDUD deleted the uuid-nix branch January 10, 2026 21:57
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.

2 participants