search

ucbi / uniform

Write less boilerplate and reuse more code in your portfolio of Elixir apps
https://hex.pm/packages/uniform
Apache License 2.0
33 stars 1 forks source link

Accept underscore in `mix uniform.eject` #27

Closed paulstatezny closed 2 years ago

paulstatezny commented 2 years ago

Would it be a more expected API to accept

mix uniform.exect some_app

Instead of this?

mix uniform.exect SomeApp

Or, should we accept both?

paulstatezny commented 2 years ago

Curious your thoughts @capitalist

paulstatezny commented 2 years ago

My thought process is that when you eject an app, you're not ejecting files with that module namespace (MyApp.*) but you're ejecting all the files in that lib folder (lib/my_app/*).

For that reason, that's how it's framed in the docs, too.

capitalist commented 2 years ago

The logic here makes sense to me 👍