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

gyp: update gyp.el to change case to cl-case #93

Merged
merged 1 commit into from
Feb 20, 2021
Merged

gyp: update gyp.el to change case to cl-case #93

merged 1 commit into from
Feb 20, 2021

Conversation

liuchong
Copy link
Contributor

@liuchong liuchong commented Jan 19, 2021 •
edited
Loading

nodejs/node-gyp#2306

‘case’ is an obsolete alias (as of 27.1); use ‘cl-case’ instead.

@cclauss
Copy link
Contributor

cclauss commented Jan 19, 2021

Do emacs lisp linters find other issues as well?

@liuchong
gyp: update gyp.el to change case to cl-case
f2b40ef 
‘case’ is an obsolete alias (as of 27.1); use ‘cl-case’ instead.
@targos targos merged commit 13d5b66 into nodejs:master Feb 20, 2021
@liuchong liuchong deleted the patch-1 branch February 20, 2021 10:17
@liuchong
Copy link
Contributor Author

🎉

@github-actions github-actions bot mentioned this pull request Mar 11, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@targos targos targos approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@liuchong @cclauss @targos