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

use the method toSymbol instead of using the attributes ns and sym #69

Closed
wants to merge 1 commit into from
Closed

use the method toSymbol instead of using the attributes ns and sym #69

wants to merge 1 commit into from

Conversation

mraveloarinjaka
Copy link

The attribute ns only exists for clojure.lang.Var but not for sci.lang.Var.
The method toSymbol can be used to create a symbol from a var and exists on clojure jvm and sci.

@mraveloarinjaka
Copy link
Author

link to issue #35

@borkdude
Copy link
Contributor

borkdude commented Dec 1, 2022

This PR should be preferred: #70 for bb compatibility

@borkdude
Copy link
Contributor

borkdude commented Dec 2, 2022

@dustingetz This can be closed now

@mraveloarinjaka
Copy link
Author

solved via 9ce258d

@mraveloarinjaka mraveloarinjaka deleted the work_with_bb branch December 3, 2022 05:42
@dustingetz
Copy link
Member

Thank you for the PR @mraveloarinjaka

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