-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Rename dynamic to ??? #4698
Copy link
Copy link
Open
Labels
Description
Elixir version
Future Elixir versions
Database and Version
All
Ecto Versions
All
Database Adapter and Versions (postgrex, myxql, etc)
All
Current behavior
Currently we define a dynamic/1 macro but:
- Elixir's type system also has a
dynamic/1and they will be confusing - Elixir's kernel may also get a
dynamic/1function, which may cause conflicts
Expected behavior
Rename it to something else. The question is what:
dynamic_queryis redundant as it is defined in Ecto.Querydynamic_fragmentfeels incompatible as fragments are stringsdynamic_exprfeels too technical? It can be because I am not a native speaker
Thoughts @greg-rychlewski?
Reactions are currently unavailable