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

Rename sigils to ~VEC and ~MAT #1114

Closed
josevalim opened this issue Mar 4, 2023 · 4 comments
Closed

Rename sigils to ~VEC and ~MAT #1114

josevalim opened this issue Mar 4, 2023 · 4 comments

Comments

@josevalim
Copy link
Collaborator

I sent a proposal for multi-letter uppercase sigils in Elixir. If accepted, we should rename our sigils to the ones above.
@dannote
Copy link
Contributor

dannote commented Apr 3, 2023

Can you explain why you don't want to keep the shorter versions? Are they confusing?

@josevalim
Copy link
Collaborator Author

josevalim commented Apr 3, 2023 •
edited

Yes, pretty much. And given we are before 1.0 and still early in the adoption process, we can afford to rename without taking a single letter sigil. Furthermore, single letter sigils are by definition limited and it increases the chances of conflicts with other community projects. Elixir (and libraries) rarely use shortcuts and sigils were an exception so far.

@zachallaun zachallaun mentioned this issue Apr 21, 2023
Closed
@josevalim
Copy link
Collaborator Author

We can now tackle this. Next Nx version can depend on Elixir v1.15.

@josevalim
Copy link
Collaborator Author

Done in 510e689.

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
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@josevalim @dannote