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

Array class notation will change toString/1, per CLJ-2833 #2303

Open
borkdude opened this issue Mar 14, 2024 · 2 comments
Open

Array class notation will change toString/1, per CLJ-2833 #2303

borkdude opened this issue Mar 14, 2024 · 2 comments
Projects
clj-kondo

Comments

@borkdude
Copy link
Member 

          fyi, this notation is going to change in next alpha to `String::1`, watch [CLJ-2833](https://clojure.atlassian.net/browse/CLJ-2833).  But `String*` could actually be a valid class name (and deftype can make such a thing), so it should be potentially valid regardless.

Originally posted by @puredanger in #2276 (comment)
@borkdude borkdude added this to Needs triage in clj-kondo via automation Mar 14, 2024
@borkdude borkdude moved this from Needs triage to High priority (next release) in clj-kondo Mar 14, 2024
@borkdude borkdude moved this from High priority (next release) to In progress in clj-kondo May 10, 2024
@borkdude borkdude moved this from In progress to High priority (next release) in clj-kondo May 10, 2024
@borkdude borkdude moved this from High priority (next release) to Medium priority (new / enhance) in clj-kondo May 17, 2024
@NoahTheDuke
Copy link
Contributor

This has been decided against, the syntax is String/1 as per CLJ-2807

@borkdude
Copy link
Member Author

Thanks for the update.

@borkdude borkdude changed the title Array class notation will change toString::1, per CLJ-2833 Array class notation will change toString/1, per CLJ-2833 May 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
clj-kondo
  
Medium priority (new / enhance)
Milestone
No milestone
Development

No branches or pull requests
2 participants
@borkdude @NoahTheDuke