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

Consider adding a namespace to #p? #24

Open
escherize opened this issue Feb 16, 2024 · 1 comment
Open

Consider adding a namespace to #p? #24

escherize opened this issue Feb 16, 2024 · 1 comment

Comments

@escherize
Copy link

Hi! Love the library, we have it installed on Metabase dev mode, and it works great.

There's a thing in the Clojure tagged literal docs about how:

Reader tags without namespace qualifiers are reserved for Clojure.

Do you think it's worth following that convention? Maybe there's a good reason not to that I'm not aware of.

@weavejester
Copy link
Owner

#p is just for debugging in development environments, so there shouldn't be an issue using reserved names. The rule to use namespace qualifiers is to ensure that third party libraries don't conflict with future Clojure changes.

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

No branches or pull requests

2 participants