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.

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

Support for GitHub and Jujitsu CLIs #1551

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

qubitz
Copy link

@qubitz qubitz commented Aug 25, 2024

Found myself using these quite often and decided to finally get around to contributing them. I'm sure there's a few standard practices that I'm missing. I primarily replicated the setup from git cli resulting in duplicate code with the csv handling. Not sure how to handle that.

I'm very ill practiced with talon. All suggestions welcome!

PS: I'd understand if jujitsu was too niche of an application to include by default with community 👌.

[jj](https://github.com/martinvonz/jj) (currently called jujitsu) is a Git-compatible VCS that is both simple and powerful.
Adds support for interacting with github through the command line with the official [GitHub CLI](https://cli.github.com/)
Copy link
Collaborator

@auscompgeek auscompgeek left a comment

Choose a reason for hiding this comment

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

I think we want to move away from CSVs for these sorts of lists, and use .talon-list instead

split
squash
status
Subcommand, Spoken form (optional)
Copy link
Collaborator

Choose a reason for hiding this comment

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

That's an interesting subcommand

Copy link
Author

Choose a reason for hiding this comment

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

😅 auto sort.

@qubitz
Copy link
Author

qubitz commented Aug 26, 2024

I think we want to move away from CSVs for these sorts of lists, and use .talon-list instead

I agree, it is a much better system, it's just that I'm a little hesitant as there's not as many examples. I'll make some attempts this weekend or next.

@nriley nriley marked this pull request as draft August 31, 2024 16:08
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