Kevin Webster
Kevin Webster
The overall strategy looks pretty good to me 👍🏼 I am a little leary about using tags for some of this data though. I don't know if we want to...
@joshuap `Notice.new` expects a map with `:context`, but it looks like `Honeybadger.notify` will jam all metadata the into the context key if its not already present: https://github.com/honeybadger-io/honeybadger-elixir/blob/24edb30cb2a2c9462bdc453c9943fa64ef9bac00/lib/honeybadger.ex#L253-L257 https://github.com/honeybadger-io/honeybadger-elixir/blob/24edb30cb2a2c9462bdc453c9943fa64ef9bac00/lib/honeybadger.ex#L428-L435
I'm not sure. I probably swing towards favoring the explicit version, just for readability sake.
Hi. Just to give an alternative solution, this is relatively easy to accomplish using just SQL. ``` javascript SELECT week, sum(passing_yds) yards FROM play_player JOIN game USING (gsis_id) JOIN player...