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

Obsolete alias #24

Open
arifer612 opened this issue Jan 4, 2024 · 0 comments · May be fixed by #25
Open

Obsolete alias #24

arifer612 opened this issue Jan 4, 2024 · 0 comments · May be fixed by #25

Comments

@arifer612
Copy link

Since version 3.7.0, helm-attrset has been obsolete. The warning raised
informs us to use instead helm-set-attr, the fully qualified function that it
references. While this is not a huge issue at this moment, it might break the
project in the future if the alias gets deprecated. To avoid this from
happening, I suggest that the alias be replaced by helm-set-attr.

I can work up a PR for this if nobody is against the idea of replacing
helm-attrset with helm-set-attr.
arifer612 added a commit to arifer612/helm-company that referenced this issue Jan 4, 2024
@arifer612
fix: replace obsolete alias with fully qualified function
27a76ec 
This commit replaces the obsolete alias `helm-attrset` with `helm-set-attr`, the
fully qualified function that it references.

Resolves Sodel-the-Vociferous#24
@arifer612 arifer612 linked a pull request Jan 4, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Replace obsolete alias with fully qualified function arifer612/helm-company
1 participant
@arifer612