revelrylabs / elixir-nodejs

An Elixir API for calling Node.js functions
MIT License
217 stars 31 forks source link

Added a `name` option #37

Closed ccapndave closed 4 years ago

ccapndave commented 4 years ago

I recently ran into a requirement where I had an umbrella where two of the apps both needed to use elixir-nodejs. This turned out not to be possible since the name and the pool name are hardcoded. Therefore this PR adds a name option which allows the name (and hence the pool) to be specified when creating the supervisor.

oohnoitz commented 4 years ago

@ccapndave just checking in to see if you were still in getting this in.

oohnoitz commented 4 years ago

Going to close this in favor of #48 which has the requested changes applied.