Skip to content

Commit

Permalink
Remove deprecations for Utils functions
Browse files Browse the repository at this point in the history
Instead of deprecating the no-longer-needed Appsignal.Utils.info/1,
Appsignal.Utils.warning/1, and Appsignal.Utils.compile_env/3, wait
until a new version of both Appsignal for Phoenix and Appsignal for
Plug are released.

That way, we won't run into a situation where a new version of
Appsignal for Elixir is released that produces deprecation warnings,
while the other libraries aren't out yet.

This allows for merging this patch without having to immediately do a
release.
  • Loading branch information
jeffkreeftmeijer committed Jun 27, 2024
1 parent 2687df4 commit 3457f08
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions lib/appsignal/utils.ex
Original file line number Diff line number Diff line change
Expand Up @@ -19,19 +19,16 @@ defmodule Appsignal.Utils do

def module_name(module), do: module |> to_string() |> module_name()

@deprecated "Use Logger.info/1 instead."
def info(message) do
require Logger
Logger.info(message)
end

@deprecated "Use Logger.warning/1 instead."
def warning(message) do
require Logger
Logger.warning(message)
end

@deprecated "Use Application.compile_env/3 instead."
defmacro compile_env(app, key, default \\ nil) do
quote do
Application.compile_env(unquote(app), unquote(key), unquote(default))
Expand Down

0 comments on commit 3457f08

Please sign in to comment.