Closed nick96 closed 1 year ago
Merging #107 (b9a1637) into main (084a430) will not change coverage. The diff coverage is
100.00%
.
:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:
I hope you don't mind me making a PR without an associated issue. I couldn't find an existing one and this seemed like a relatively easy fix.
The documentation for
:telemetry.attach/4
states that function capture should be used for the event handler, rather than local function capture for performance reasons. Using local function capture also leads to warnings like:Unfortunately, to allow using function capture, I've had to make
handle_event/4
a public function.