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.

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

Allow specifying an optional name for import-fn and import-macro #9

Merged
merged 1 commit into from
May 26, 2013

Conversation

dakrone
Copy link
Contributor

@dakrone dakrone commented May 21, 2013

I apologize for the whitespace cleanup in this, if it bothers you I can try to do the PR without it.

Adds the ability to specify an optional name for importing something with import-fn and import-macro. Which allows using import-* for aliasing.

@ztellman
Copy link
Collaborator

I use a weird indenting scheme (two spaces everywhere), it doesn't bother me, but it will naturally return to its previous state once I touch the file again.

This looks good, but I was just about to switch this over to the MIT license, so that it's easier for people to cut-and-paste stuff from here if they don't want it as a dependency. Do you mind if I swap that out from underneath you before merging your code?

@dakrone
Copy link
Contributor Author

dakrone commented May 21, 2013

Go for it, that sounds good to me.

ztellman added a commit that referenced this pull request May 26, 2013
Allow specifying an optional name for import-fn and import-macro
@ztellman ztellman merged commit 739821c into clj-commons:master May 26, 2013
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.

2 participants