Incorrect Exception modules should be named consistently

[joshk]

Environment

• Credo version (mix credo -v): 1.7.11
• Erlang/Elixir version (elixir -v): 1.18.0
• Operating system: Mac 15.1.1

What were you trying to do?

defmodule NervesHub.Errors.Unauthorized do
  defexception message: "not found", plug_status: 404
end

defmodule NervesHubWeb.NotFoundError do
  defexception message: "not found", plug_status: 404
end

defmodule NervesHubWeb.UnauthorizedError do
  defexception message: "forbidden", plug_status: 401
end

Expected outcome

No warnings.

Actual outcome

  Consistency
┃
┃ [C] ↗ Exception modules should be named consistently. It seems your strategy is to prefix them with `Unauthorized`, but
┃       `NervesHubWeb.NotFoundError` does not follow that convention.
┃       lib/nerves_hub_web/errors.ex:5:11 #(NervesHubWeb.NotFoundError)

[joshk]

Although I do regard this as a bug, I did fix this by reworking our code a little to remove the duplicate Unauthorized exception.