romul / newrelic.ex

NewRelic agent for Elixir
MIT License
64 stars 19 forks source link

Remove compilation errors for Phoenix/Ecto not being included #15

Closed sb8244 closed 2 years ago

sb8244 commented 5 years ago

WIP for #14

There are still compilation errors with this, because the functions don't exist if the dependencies aren't there. I've solved this in the past with quoted AST in the library that gets used into a module, but that probably isn't desired here. Going to think a bit more about how to get rid of these compilation errors if ecto/phoenix aren't included.

sb8244 commented 5 years ago

Sorry for the tags getting created from this PR. My git client pushed up all tags without me telling it to. Deleted them, and will work on fork in separate clone.