Skip to content

Feature: insert issue reference on <CR> - #33

Merged
sbulav merged 1 commit into
nvim-telescope:masterfrom
cljoly:master
Jan 18, 2022
Merged

sbulav merged 1 commit into
nvim-telescope:masterfrom
cljoly:master

Conversation

@cljoly

@cljoly cljoly commented Jan 3, 2022

Copy link
Copy Markdown
Contributor

Currently, <CR> is set to “close Telescope”. There is already a
generic “cancel” mapping for that. I find more useful to just insert the
selected issue in the buffer we come from.

This allows to insert references to GitHub issues in commit messages,
code comments…

@sbulav

sbulav commented Jan 10, 2022

Copy link
Copy Markdown
Collaborator

Hi @cljoly

I agree that it would be more convenient.
I've merged my fork with a lot of changes so please resolve conflicts before I can merge your PR.

@cljoly

cljoly commented Jan 11, 2022 •

Copy link
Copy Markdown
Contributor Author

Hi @sbulav , I'll look into rebasing and fixing conflicts in the next few days. Thanks for your work on this project!

@cljoly

cljoly commented Jan 14, 2022

Copy link
Copy Markdown
Contributor Author

Alright, it works on my computer, this should be ready if you wanna give it a try.

@sbulav sbulav left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Overall looks good, but please stick with the same approach for inserting text as we use for other actions.

Comment thread lua/telescope/_extensions/gh_actions.lua Outdated
Currently, `<CR>` is set to “close Telescope”. There is already a
generic “cancel” mapping for that. I find more useful to just insert the
selected issue in the buffer we come from.

This allows to insert references to GitHub issues in commit messages,
code comments…
@sbulav

sbulav commented Jan 18, 2022

Copy link
Copy Markdown
Collaborator

Merged. Thanks for your contribution!

@sbulav
sbulav merged commit e4783c7 into nvim-telescope:master Jan 18, 2022
@cljoly

cljoly commented Jan 19, 2022

Copy link
Copy Markdown
Contributor Author

Awesome, thanks!

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