Skip to content

Rename dynamic to ??? #4698

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?

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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