Skip to content

feat: Add o as a command to open up a pr on github - #199

Open
basola21 wants to merge 5 commits into
Cretezy:mainfrom
basola21:main
Open

basola21 wants to merge 5 commits into
Cretezy:mainfrom
basola21:main

Conversation

@basola21

Copy link
Copy Markdown

Hey, I have been having fun with jj and your work here is amazing, anyhow recently I have added some of my changes, I do not know if you may like it (it is a little opinionated) , anyhow it is just another command to use o as a pr opener on github, nothing crazy there

  • please ignore the previous two commits, I was just playing around, if you want this as a feature I could clean it up to use config keys -> better for users that are not using github.

thanks for this amazing tool

@peso

peso commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

I like the idea of getting more help to do frequent stuff. I think you should split this into two PR: One for AI assistance and one for Github. Also, it should be more configurable - don't hardcode the URL to your AI.

@basola21

Copy link
Copy Markdown
Author

I like the idea of getting more help to do frequent stuff. I think you should split this into two PR: One for AI assistance and one for Github. Also, it should be more configurable - don't hardcode the URL to your AI.

I was primarily testing the idea to see if you liked it. I discovered that I frequently use Ctrl+O to open PR's, and I didn’t use the AI-assisted feature at all, so I’ll likely remove it.

@dotdash

dotdash commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

FWIW, lazyjj is currently basically unmaintained. We have an active fork at https://github.com/blazingjj/blazingjj

@basola21
basola21 marked this pull request as ready for review September 11, 2026 11:26
@basola21

Copy link
Copy Markdown
Author

@dotdash thanks I will open up the pr there then

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.

3 participants