Address credo issues in touched files

Author: zoldar

lib/mix/tasks/create_free_subscription.ex

@@ -1,4 +1,6 @@
 defmodule Mix.Tasks.CreateFreeSubscription do
+  @moduledoc false
+
   use Mix.Task
   use Plausible.Repo
 

lib/mix/tasks/generate_referrer_favicons.ex

@@ -1,4 +1,6 @@
 defmodule Mix.Tasks.GenerateReferrerFavicons do
+  @moduledoc false
+
   use Mix.Task
   use Plausible.Repo
 

lib/plausible/session/cache_store.ex

@@ -1,4 +1,8 @@
 defmodule Plausible.Session.CacheStore do
+  @moduledoc """
+  Session management on the basis of incoming events.
+  """
+
   alias Plausible.Session.WriteBuffer
 
   @lock_timeout 1000

lib/plausible/stats/query.ex

@@ -1,4 +1,8 @@
 defmodule Plausible.Stats.Query do
+  @moduledoc """
+  Struct storing stats query parameters and configuration.
+  """
+
   use Plausible
 
   defstruct utc_time_range: nil,

lib/plausible_release.ex

@@ -1,4 +1,9 @@
 defmodule Plausible.Release do
+  @moduledoc """
+  Utility functions used by release scripts for routine tasks, 
+  like migrations. 
+  """
+
   use Plausible
   use Plausible.Repo
 

lib/plausible_web.ex

@@ -1,4 +1,6 @@
 defmodule PlausibleWeb do
+  @moduledoc false
+
   use Plausible
 
   def live_view(opts \\ []) do

lib/workers/schedule_email_reports.ex

@@ -1,4 +1,7 @@
 defmodule Plausible.Workers.ScheduleEmailReports do
+  @moduledoc """
+  Job scheduling dispatch of weekly and monthly email reports.
+  """
   use Plausible.Repo
   use Oban.Worker, queue: :schedule_email_reports
   alias Plausible.Workers.SendEmailReport

lib/workers/send_site_setup_emails.ex

@@ -1,4 +1,8 @@
 defmodule Plausible.Workers.SendSiteSetupEmails do
+  @moduledoc """
+  Job sending site setup emails.
+  """
+
   use Plausible.Repo
   use Oban.Worker, queue: :site_setup_emails
 

lib/workers/send_trial_notifications.ex

@@ -1,4 +1,7 @@
 defmodule Plausible.Workers.SendTrialNotifications do
+  @moduledoc """
+  Job sending trial end notification emails.
+  """
   use Plausible.Repo
 
   use Oban.Worker,
@@ -20,30 +23,23 @@ defmodule Plausible.Workers.SendTrialNotifications do
           order_by: t.inserted_at,
           preload: [owners: o, billing_members: bm]
       )
+      |> Enum.filter(&Teams.has_active_sites?/1)
 
     for team <- teams do
       recipients = team.owners ++ team.billing_members
 
       case Date.diff(team.trial_expiry_date, Date.utc_today()) do
         7 ->
-          if Teams.has_active_sites?(team) do
-            send_one_week_reminder(recipients, team)
-          end
+          send_one_week_reminder(recipients, team)
 
         1 ->
-          if Teams.has_active_sites?(team) do
-            send_tomorrow_reminder(recipients, team)
-          end
+          send_tomorrow_reminder(recipients, team)
 
         0 ->
-          if Teams.has_active_sites?(team) do
-            send_today_reminder(recipients, team)
-          end
+          send_today_reminder(recipients, team)
 
         -1 ->
-          if Teams.has_active_sites?(team) do
-            send_over_reminder(recipients, team)
-          end
+          send_over_reminder(recipients, team)
 
         _ ->
           nil