Skip to content

[Resolve #366] Add cljr-rename-ns-alias. #440

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

Closed
wants to merge 1 commit into from

Conversation

anthonygalea
Copy link
Contributor

@anthonygalea anthonygalea commented May 17, 2019

Refactoring works but I have an issue with the Without :as scenario. I think it's related to the action chain but not sure how to fix it. My goal there is to ensure that if a rename is made for an alias which is not in the require the file is untouched.

@anthonygalea
Copy link
Contributor Author

Ok ignore my last comment. I see I was wrongly using user-error instead of message.

@expez
Copy link
Member

expez commented May 18, 2019

Hi, thanks for taking an interest @anthonygalea! This is great stuff! 👍

Since this is elisp only, do you think we should put this in clojure-mode @bbatsov?

@anthonygalea
Copy link
Contributor Author

Thanks @expez, @bbatsov if this should go into clojure-mode just let me know, happy to look into it.

@bbatsov
Copy link
Member

bbatsov commented Jun 18, 2019

@anthonygalea Sorry about the slow response! Let's add this directly to clojure-mode!

@anthonygalea
Copy link
Contributor Author

No problem @bbatsov. Opened another pull request in clojure-mode so I'm closing this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants