-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
Jump around jump jump #36
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
New command to swap between the feature and its tests. Signed-off-by: Psionik K <73710933+psionic-k@users.noreply.github.com>
Signed-off-by: Psionik K <73710933+psionic-k@users.noreply.github.com>
Signed-off-by: Psionik K <73710933+psionic-k@users.noreply.github.com>
See manual update for notes Signed-off-by: Psionik K <73710933+psionic-k@users.noreply.github.com>
Got confused. Intentionally didn't add test feature or project package to discourage the habit, but then I really wanted to jump to tests when looking at the menu. Signed-off-by: Psionik K <73710933+psionic-k@users.noreply.github.com>
Signed-off-by: Psionik K <73710933+psionic-k@users.noreply.github.com>
Signed-off-by: Psionik K <73710933+psionic-k@users.noreply.github.com>
I forget every freaking time. Good place to add a transient wizard ^_^ Signed-off-by: Psionik K <73710933+psionic-k@users.noreply.github.com>
Waterboard everything Signed-off-by: Psionik K <73710933+psionic-k@users.noreply.github.com>
psionic-k
force-pushed
the
jump-around-jump-jump
branch
4 times, most recently
from
October 29, 2023 14:57
70e8fa3
to
c6f6796
Compare
Since the only reference is inside of a hook, and only as a default value, do we actually require this? It's going to usually succeed at runtime. I can add the requirement to the test. I do anticipate doing more magit integration in the future, such as jumping to tests form magit buffers (should just work actually) Anyway. Everyone has it. No harm, no foul. Signed-off-by: Psionik K <73710933+psionic-k@users.noreply.github.com>
Signed-off-by: Psionik K <73710933+psionic-k@users.noreply.github.com>
psionic-k
force-pushed
the
jump-around-jump-jump
branch
from
October 29, 2023 15:01
c6f6796
to
6b6650b
Compare
Signed-off-by: Psionik K <73710933+psionic-k@users.noreply.github.com>
psionic-k
force-pushed
the
jump-around-jump-jump
branch
from
October 29, 2023 15:48
cc2a236
to
f22ffd2
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Introducing⚠️
erk-jump-defs
anderk-jump-features
And verifying that CI still works because releases must pass!