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

Add support for opening counsel-git-grep in other window #3044

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

flooose
Copy link

@flooose flooose commented Apr 25, 2024

Adds functionality described in related issue

Copy link
Collaborator

@basil-conto basil-conto left a comment

Choose a reason for hiding this comment

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

Thanks! Works nicely.

See some minor style/logic suggestions below.

counsel.el Outdated Show resolved Hide resolved
counsel.el Outdated Show resolved Hide resolved
counsel.el Outdated Show resolved Hide resolved
counsel.el Outdated Show resolved Hide resolved
counsel.el Outdated Show resolved Hide resolved
@basil-conto basil-conto added the waiting-for-ca Waiting for author's copyright assignment label Apr 27, 2024
New regex:"\\`\\(.*\\):\\([[:digit:]]+\\):\\(.*\\)\\'"

Old regex: "\\`\\(.*?\\):\\([0-9]+\\):\\(.*\\)\\'"
Copy link
Collaborator

@basil-conto basil-conto left a comment

Choose a reason for hiding this comment

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

Thanks, we're almost there! The only real issue is the use of rx-define, which is not sufficiently backward compatible (one day we'll bump Counsel's minimum Emacs version, but not today :).

counsel.el Outdated Show resolved Hide resolved
counsel.el Outdated Show resolved Hide resolved
counsel.el Outdated Show resolved Hide resolved
flooose and others added 4 commits May 1, 2024 20:15
* counsel.el (counsel-git-grep-function): Simplify.
(counsel--git-grep-file-and-line-number): Rename...
(counsel--git-grep-file-and-line): ...to this, splitting long lines.
(counsel--git-grep-go-to-location): Rename...
(counsel--git-grep-visit): ...to this, incorporating more DRY.
(counsel-git-grep-action, counsel-git-grep-action-other-window):
Adapt accordingly (abo-abo#3044).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement waiting-for-ca Waiting for author's copyright assignment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants