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.

Sign up for GitHub

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

Jump to bottom

adds a written command to clear the jumplist. #6251

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

adds a written command to clear the jumplist. #6251

wants to merge 2 commits into from

Conversation

Bruce-Hopkins
Copy link
Contributor

Fixes #4757. Adds the typed command :clear-jumplist to completely clear the jumplist.

@grv07
Copy link
Contributor

grv07 commented May 27, 2023

@Bruce-Hopkins run the following commands to fix these checks

  1. cargo fmt
  2. cargo xtask docgen
    https://github.com/helix-editor/helix/blob/master/docs/CONTRIBUTING.md#auto-generated-documentation

@Bruce-Hopkins
Copy link
Contributor Author

I just did, still don't know why the checks are failing

@gabydd
Copy link
Member

gabydd commented May 27, 2023

On master TypeableCommand has changed see https://github.com/helix-editor/helix/blob/master/helix-term/src/commands/typed.rs#L2234-L2240 for an example of how to set the completer which has changed to be called signature. You probably just need to rebase on master and change that.

@the-mikedavis the-mikedavis mentioned this pull request Jun 4, 2023
Closed
@workingj
Copy link
Contributor

workingj commented Oct 8, 2023

hi there, any progress here possible in the near future?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Clear jumplist
4 participants
@Bruce-Hopkins @grv07 @gabydd @workingj