Skip to content

Rename dynamic to ??? #4698

@josevalim

Description

@josevalim

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:

  1. Elixir's type system also has a dynamic/1 and they will be confusing
  2. Elixir's kernel may also get a dynamic/1 function, which may cause conflicts

Expected behavior

Rename it to something else. The question is what:

  1. dynamic_query is redundant as it is defined in Ecto.Query
  2. dynamic_fragment feels incompatible as fragments are strings
  3. dynamic_expr feels too technical? It can be because I am not a native speaker

Thoughts @greg-rychlewski?

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions