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

Multi word abbreviations (#32; v5) #50

Merged
merged 26 commits into from
Dec 21, 2021
Merged

Multi word abbreviations (#32; v5) #50

merged 26 commits into from
Dec 21, 2021

Conversation

olets
Copy link
Owner

@olets olets commented Dec 21, 2021

For discussion see #32

Abbreviations can be more than one word

% abbr "git cp"="git cherry-pick"
% git cp<space>
# expands inline to `git cherry-pick`!

% abbr g=git
% g<space>fp<space>
# expands inline to `git cherry-pick`!

@olets
feat(expand): try all substrings from longest to shortest
d7bce29
@olets
feat(add): do not error on multi-word abbreviations
ebe5273
@olets
chore(job): minor grammar correction
87ee208
@olets
style(add): check for negatives when deciding to throw
7214435
@olets
tests(multi-word abbreviations): new skipped tests
491dc2f
@olets
feat(add): support multi-word abbreviations
90a578c
@olets
chore(add): remove apparently unnecessary code. safe??
564f851
@olets
feat(util_sync_user): support multiword. with question...
6129ddf 
Should '${(qqq)abbreviation}' be '${(qqq)${(Q)abbreviation}}'?
@olets
feat(_abbr_cmd_expansion): support multiword. with question...
8a9bd3a 
Should '${(qqq)abbreviation}' be '${(qqq)${(Q)abbreviation}}'?
@olets
feat(erase, expand): support multi-word
940aedb
@olets
feat(erase,add): remove some possibly unnecessary quoting
4f92a03
@olets
tests(multi-word): do not skip any tests
bd34f9a
@olets
fix(util_sync_user): abbreviations survive sync
a65b279
@olets
fix(util_sync_user): remove possibly unnecessary quoting
2da589a
@olets
tests(all): account for changed quote levels
6093675
@olets
fix(add): do not lose quotation marks
6273b2e
@olets
fix(logs): correct quote level
1cc4110
@olets
fix(erase): erases abbreviation (quote level correction)
dfc5c2a
@olets
tests(all): correct quote levels
4379e02
@olets
feat(rename): can rename (quote level correction)
df36b15
@olets
tests(rename): passes (correct test typo)
b68d4f7
@olets
chore(comments): correct typo
9ef743d
@olets
feat(expansion): support expanding global abbreviations
0a23e09
@olets
tests(git import): switch to using 'abbr expand' + correct typo
cfbdd6c
@olets
tests(expand): new: global, session, and session global
59d268b
@olets
docs(add): multi-word abbreviations are supported
565c17b
@olets olets merged commit a05f1c7 into v5 Dec 21, 2021
@olets olets deleted the multi-word-abbreviations branch December 21, 2021 03:04
@olets olets changed the title Multi word abbreviations (#32) Multi word abbreviations (#32; v5) Dec 21, 2021
@olets olets mentioned this pull request Mar 3, 2023
Open
2 tasks
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
zsh-abbr v5
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@olets