diff --git a/.credo.exs b/.credo.exs new file mode 100644 index 00000000..539d213b --- /dev/null +++ b/.credo.exs @@ -0,0 +1,216 @@ +# This file contains the configuration for Credo and you are probably reading +# this after creating it with `mix credo.gen.config`. +# +# If you find anything wrong or unclear in this file, please report an +# issue on GitHub: https://github.com/rrrene/credo/issues +# +%{ + # + # You can have as many configs as you like in the `configs:` field. + configs: [ + %{ + # + # Run any config using `mix credo -C `. If no config name is given + # "default" is used. + # + name: "default", + # + # These are the files included in the analysis: + files: %{ + # + # You can give explicit globs or simply directories. + # In the latter case `**/*.{ex,exs}` will be used. + # + included: [ + "lib/", + "src/", + "test/", + "web/", + "apps/*/lib/", + "apps/*/src/", + "apps/*/test/", + "apps/*/web/" + ], + excluded: [~r"/_build/", ~r"/deps/", ~r"/node_modules/"] + }, + # + # Load and configure plugins here: + # + plugins: [], + # + # If you create your own checks, you must specify the source files for + # them here, so they can be loaded by Credo before running the analysis. + # + requires: [], + # + # If you want to enforce a style guide and need a more traditional linting + # experience, you can change `strict` to `true` below: + # + strict: false, + # + # To modify the timeout for parsing files, change this value: + # + parse_timeout: 5000, + # + # If you want to use uncolored output by default, you can change `color` + # to `false` below: + # + color: true, + # + # You can customize the parameters of any check by adding a second element + # to the tuple. + # + # To disable a check put `false` as second element: + # + # {Credo.Check.Design.DuplicatedCode, false} + # + checks: %{ + enabled: [ + # + ## Consistency Checks + # + {Credo.Check.Consistency.ExceptionNames, []}, + {Credo.Check.Consistency.LineEndings, []}, + {Credo.Check.Consistency.ParameterPatternMatching, []}, + {Credo.Check.Consistency.SpaceAroundOperators, []}, + {Credo.Check.Consistency.SpaceInParentheses, []}, + {Credo.Check.Consistency.TabsOrSpaces, []}, + + # + ## Design Checks + # + # You can customize the priority of any check + # Priority values are: `low, normal, high, higher` + # + {Credo.Check.Design.AliasUsage, + [priority: :low, if_nested_deeper_than: 2, if_called_more_often_than: 0]}, + {Credo.Check.Design.TagFIXME, []}, + # You can also customize the exit_status of each check. + # If you don't want TODO comments to cause `mix credo` to fail, just + # set this value to 0 (zero). + # + {Credo.Check.Design.TagTODO, [exit_status: 2]}, + + # + ## Readability Checks + # + {Credo.Check.Readability.AliasOrder, []}, + {Credo.Check.Readability.FunctionNames, []}, + {Credo.Check.Readability.LargeNumbers, []}, + {Credo.Check.Readability.MaxLineLength, [priority: :low, max_length: 120]}, + {Credo.Check.Readability.ModuleAttributeNames, []}, + {Credo.Check.Readability.ModuleDoc, []}, + {Credo.Check.Readability.ModuleNames, []}, + {Credo.Check.Readability.ParenthesesInCondition, []}, + {Credo.Check.Readability.ParenthesesOnZeroArityDefs, []}, + {Credo.Check.Readability.PipeIntoAnonymousFunctions, []}, + {Credo.Check.Readability.PredicateFunctionNames, []}, + {Credo.Check.Readability.PreferImplicitTry, []}, + {Credo.Check.Readability.RedundantBlankLines, []}, + {Credo.Check.Readability.Semicolons, []}, + {Credo.Check.Readability.SpaceAfterCommas, []}, + {Credo.Check.Readability.StringSigils, []}, + {Credo.Check.Readability.TrailingBlankLine, []}, + {Credo.Check.Readability.TrailingWhiteSpace, []}, + {Credo.Check.Readability.UnnecessaryAliasExpansion, []}, + {Credo.Check.Readability.VariableNames, []}, + {Credo.Check.Readability.WithSingleClause, []}, + + # + ## Refactoring Opportunities + # + {Credo.Check.Refactor.Apply, []}, + {Credo.Check.Refactor.CondStatements, []}, + {Credo.Check.Refactor.CyclomaticComplexity, []}, + {Credo.Check.Refactor.FilterCount, []}, + {Credo.Check.Refactor.FilterFilter, []}, + {Credo.Check.Refactor.FunctionArity, []}, + {Credo.Check.Refactor.LongQuoteBlocks, []}, + {Credo.Check.Refactor.MapJoin, []}, + {Credo.Check.Refactor.MatchInCondition, []}, + {Credo.Check.Refactor.NegatedConditionsInUnless, []}, + {Credo.Check.Refactor.NegatedConditionsWithElse, []}, + {Credo.Check.Refactor.Nesting, []}, + {Credo.Check.Refactor.RedundantWithClauseResult, []}, + {Credo.Check.Refactor.RejectReject, []}, + {Credo.Check.Refactor.UnlessWithElse, []}, + {Credo.Check.Refactor.WithClauses, []}, + + # + ## Warnings + # + {Credo.Check.Warning.ApplicationConfigInModuleAttribute, []}, + {Credo.Check.Warning.BoolOperationOnSameValues, []}, + {Credo.Check.Warning.Dbg, []}, + {Credo.Check.Warning.ExpensiveEmptyEnumCheck, []}, + {Credo.Check.Warning.IExPry, []}, + {Credo.Check.Warning.IoInspect, []}, + {Credo.Check.Warning.MissedMetadataKeyInLoggerConfig, []}, + {Credo.Check.Warning.OperationOnSameValues, []}, + {Credo.Check.Warning.OperationWithConstantResult, []}, + {Credo.Check.Warning.RaiseInsideRescue, []}, + {Credo.Check.Warning.SpecWithStruct, []}, + {Credo.Check.Warning.UnsafeExec, []}, + {Credo.Check.Warning.UnusedEnumOperation, []}, + {Credo.Check.Warning.UnusedFileOperation, []}, + {Credo.Check.Warning.UnusedKeywordOperation, []}, + {Credo.Check.Warning.UnusedListOperation, []}, + {Credo.Check.Warning.UnusedPathOperation, []}, + {Credo.Check.Warning.UnusedRegexOperation, []}, + {Credo.Check.Warning.UnusedStringOperation, []}, + {Credo.Check.Warning.UnusedTupleOperation, []}, + {Credo.Check.Warning.WrongTestFileExtension, []} + ], + disabled: [ + # + # Checks scheduled for next check update (opt-in for now, just replace `false` with `[]`) + + # + # Controversial and experimental checks (opt-in, just move the check to `:enabled` + # and be sure to use `mix credo --strict` to see low priority checks) + # + {Credo.Check.Consistency.MultiAliasImportRequireUse, []}, + {Credo.Check.Consistency.UnusedVariableNames, []}, + {Credo.Check.Design.DuplicatedCode, []}, + {Credo.Check.Design.SkipTestWithoutComment, []}, + {Credo.Check.Readability.AliasAs, []}, + {Credo.Check.Readability.BlockPipe, []}, + {Credo.Check.Readability.ImplTrue, []}, + {Credo.Check.Readability.MultiAlias, []}, + {Credo.Check.Readability.NestedFunctionCalls, []}, + {Credo.Check.Readability.OneArityFunctionInPipe, []}, + {Credo.Check.Readability.OnePipePerLine, []}, + {Credo.Check.Readability.SeparateAliasRequire, []}, + {Credo.Check.Readability.SingleFunctionToBlockPipe, []}, + {Credo.Check.Readability.SinglePipe, []}, + {Credo.Check.Readability.Specs, []}, + {Credo.Check.Readability.StrictModuleLayout, []}, + {Credo.Check.Readability.WithCustomTaggedTuple, []}, + {Credo.Check.Refactor.ABCSize, []}, + {Credo.Check.Refactor.AppendSingleItem, []}, + {Credo.Check.Refactor.DoubleBooleanNegation, []}, + {Credo.Check.Refactor.FilterReject, []}, + {Credo.Check.Refactor.IoPuts, []}, + {Credo.Check.Refactor.MapMap, []}, + {Credo.Check.Refactor.ModuleDependencies, []}, + {Credo.Check.Refactor.NegatedIsNil, []}, + {Credo.Check.Refactor.PassAsyncInTestCases, []}, + {Credo.Check.Refactor.PipeChainStart, []}, + {Credo.Check.Refactor.RejectFilter, []}, + {Credo.Check.Refactor.VariableRebinding, []}, + {Credo.Check.Warning.LazyLogging, []}, + {Credo.Check.Warning.LeakyEnvironment, []}, + {Credo.Check.Warning.MapGetUnsafePass, []}, + {Credo.Check.Warning.MixEnv, []}, + {Credo.Check.Warning.UnsafeToAtom, []} + + # {Credo.Check.Refactor.MapInto, []}, + + # + # Custom checks can be created using `mix credo.gen.check`. + # + ] + } + } + ] +} diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index dc2b6406..c9618a59 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -68,6 +68,9 @@ jobs: - name: Compiles without warnings run: mix compile --warnings-as-errors + - name: Check Code Style + run: mix credo --strict + - name: Check Formatting run: mix format --check-formatted diff --git a/lib/radiator/accounts.ex b/lib/radiator/accounts.ex index 8f207dc5..71898853 100644 --- a/lib/radiator/accounts.ex +++ b/lib/radiator/accounts.ex @@ -6,7 +6,7 @@ defmodule Radiator.Accounts do import Ecto.Query, warn: false alias Radiator.Repo - alias Radiator.Accounts.{User, UserToken, UserNotifier} + alias Radiator.Accounts.{User, UserNotifier, UserToken} ## Database getters diff --git a/lib/radiator/accounts/user.ex b/lib/radiator/accounts/user.ex index 4aeca846..3412f376 100644 --- a/lib/radiator/accounts/user.ex +++ b/lib/radiator/accounts/user.ex @@ -1,4 +1,7 @@ defmodule Radiator.Accounts.User do + @moduledoc """ + The user model. + """ use Ecto.Schema import Ecto.Changeset diff --git a/lib/radiator/accounts/user_notifier.ex b/lib/radiator/accounts/user_notifier.ex index 0deda5c4..86387652 100644 --- a/lib/radiator/accounts/user_notifier.ex +++ b/lib/radiator/accounts/user_notifier.ex @@ -1,4 +1,7 @@ defmodule Radiator.Accounts.UserNotifier do + @moduledoc """ + Handles notification emails for users. + """ import Swoosh.Email alias Radiator.Mailer diff --git a/lib/radiator/accounts/user_token.ex b/lib/radiator/accounts/user_token.ex index 03c0a8d8..487d6bc0 100644 --- a/lib/radiator/accounts/user_token.ex +++ b/lib/radiator/accounts/user_token.ex @@ -1,4 +1,7 @@ defmodule Radiator.Accounts.UserToken do + @moduledoc """ + Token handling for user sessions. + """ use Ecto.Schema import Ecto.Query alias Radiator.Accounts.UserToken diff --git a/lib/radiator/mailer.ex b/lib/radiator/mailer.ex index 100e15a8..2ae4dad1 100644 --- a/lib/radiator/mailer.ex +++ b/lib/radiator/mailer.ex @@ -1,3 +1,4 @@ defmodule Radiator.Mailer do + @moduledoc false use Swoosh.Mailer, otp_app: :radiator end diff --git a/lib/radiator/outline.ex b/lib/radiator/outline.ex index 20cbe7c9..eac8b317 100644 --- a/lib/radiator/outline.ex +++ b/lib/radiator/outline.ex @@ -2,12 +2,11 @@ defmodule Radiator.Outline do @moduledoc """ The Outline context. """ - import Ecto.Query, warn: false alias Phoenix.PubSub - alias Radiator.Repo alias Radiator.Outline.Node + alias Radiator.Repo @topic "outline" diff --git a/lib/radiator/outline/node.ex b/lib/radiator/outline/node.ex index f614772d..5f716919 100644 --- a/lib/radiator/outline/node.ex +++ b/lib/radiator/outline/node.ex @@ -1,4 +1,8 @@ defmodule Radiator.Outline.Node do + @moduledoc """ + The node model which represents a single node in the outline. + Currenty there is no concept of a tree + """ use Ecto.Schema import Ecto.Changeset diff --git a/lib/radiator_web/components/core_components.ex b/lib/radiator_web/components/core_components.ex index 42b63c3c..ede24885 100644 --- a/lib/radiator_web/components/core_components.ex +++ b/lib/radiator_web/components/core_components.ex @@ -16,7 +16,9 @@ defmodule RadiatorWeb.CoreComponents do """ use Phoenix.Component + alias Phoenix.HTML.Form alias Phoenix.LiveView.JS + import RadiatorWeb.Gettext @doc """ @@ -303,7 +305,7 @@ defmodule RadiatorWeb.CoreComponents do def input(%{type: "checkbox"} = assigns) do assigns = assign_new(assigns, :checked, fn -> - Phoenix.HTML.Form.normalize_value("checkbox", assigns[:value]) + Form.normalize_value("checkbox", assigns[:value]) end) ~H""" diff --git a/lib/radiator_web/components/layouts.ex b/lib/radiator_web/components/layouts.ex index 52a6c44b..4e2fe97c 100644 --- a/lib/radiator_web/components/layouts.ex +++ b/lib/radiator_web/components/layouts.ex @@ -1,4 +1,5 @@ defmodule RadiatorWeb.Layouts do + @moduledoc false use RadiatorWeb, :html embed_templates "layouts/*" diff --git a/lib/radiator_web/telemetry.ex b/lib/radiator_web/telemetry.ex index 4cb0b971..7ccd2c56 100644 --- a/lib/radiator_web/telemetry.ex +++ b/lib/radiator_web/telemetry.ex @@ -1,4 +1,5 @@ defmodule RadiatorWeb.Telemetry do + @moduledoc false use Supervisor import Telemetry.Metrics diff --git a/lib/radiator_web/user_auth.ex b/lib/radiator_web/user_auth.ex index ff2c37fe..3e20cc1a 100644 --- a/lib/radiator_web/user_auth.ex +++ b/lib/radiator_web/user_auth.ex @@ -1,4 +1,7 @@ defmodule RadiatorWeb.UserAuth do + @moduledoc """ + This module provides functions to log in and log out users. + """ use RadiatorWeb, :verified_routes import Plug.Conn diff --git a/mix.exs b/mix.exs index b4e795be..4580ac53 100644 --- a/mix.exs +++ b/mix.exs @@ -33,6 +33,7 @@ defmodule Radiator.MixProject do defp deps do [ {:argon2_elixir, "~> 3.0"}, + {:credo, "~> 1.6", only: [:dev, :test], runtime: false}, {:dns_cluster, "~> 0.1.1"}, {:ecto_sql, "~> 3.10"}, {:esbuild, "~> 0.8", runtime: Mix.env() == :dev}, @@ -42,12 +43,12 @@ defmodule Radiator.MixProject do {:gettext, "~> 0.20"}, {:jason, "~> 1.2"}, {:mix_test_watch, "~> 1.0", only: [:dev, :test], runtime: false}, + {:phoenix, "~> 1.7.10"}, {:phoenix_ecto, "~> 4.4"}, {:phoenix_html, "~> 3.3"}, {:phoenix_live_dashboard, "~> 0.8.2"}, {:phoenix_live_reload, "~> 1.2", only: :dev}, {:phoenix_live_view, "~> 0.20.1"}, - {:phoenix, "~> 1.7.10"}, {:plug_cowboy, "~> 2.5"}, {:postgrex, ">= 0.0.0"}, {:swoosh, "~> 1.3"}, diff --git a/mix.lock b/mix.lock index 39ca9ae6..74e51b0b 100644 --- a/mix.lock +++ b/mix.lock @@ -1,10 +1,12 @@ %{ "argon2_elixir": {:hex, :argon2_elixir, "3.2.1", "f47740bf9f2a39ffef79ba48eb25dea2ee37bcc7eadf91d49615591d1a6fce1a", [:make, :mix], [{:comeonin, "~> 5.3", [hex: :comeonin, repo: "hexpm", optional: false]}, {:elixir_make, "~> 0.6", [hex: :elixir_make, repo: "hexpm", optional: false]}], "hexpm", "a813b78217394530b5fcf4c8070feee43df03ffef938d044019169c766315690"}, + "bunt": {:hex, :bunt, "0.2.1", "e2d4792f7bc0ced7583ab54922808919518d0e57ee162901a16a1b6664ef3b14", [:mix], [], "hexpm", "a330bfb4245239787b15005e66ae6845c9cd524a288f0d141c148b02603777a5"}, "castore": {:hex, :castore, "1.0.4", "ff4d0fb2e6411c0479b1d965a814ea6d00e51eb2f58697446e9c41a97d940b28", [:mix], [], "hexpm", "9418c1b8144e11656f0be99943db4caf04612e3eaecefb5dae9a2a87565584f8"}, "comeonin": {:hex, :comeonin, "5.4.0", "246a56ca3f41d404380fc6465650ddaa532c7f98be4bda1b4656b3a37cc13abe", [:mix], [], "hexpm", "796393a9e50d01999d56b7b8420ab0481a7538d0caf80919da493b4a6e51faf1"}, "cowboy": {:hex, :cowboy, "2.10.0", "ff9ffeff91dae4ae270dd975642997afe2a1179d94b1887863e43f681a203e26", [:make, :rebar3], [{:cowlib, "2.12.1", [hex: :cowlib, repo: "hexpm", optional: false]}, {:ranch, "1.8.0", [hex: :ranch, repo: "hexpm", optional: false]}], "hexpm", "3afdccb7183cc6f143cb14d3cf51fa00e53db9ec80cdcd525482f5e99bc41d6b"}, "cowboy_telemetry": {:hex, :cowboy_telemetry, "0.4.0", "f239f68b588efa7707abce16a84d0d2acf3a0f50571f8bb7f56a15865aae820c", [:rebar3], [{:cowboy, "~> 2.7", [hex: :cowboy, repo: "hexpm", optional: false]}, {:telemetry, "~> 1.0", [hex: :telemetry, repo: "hexpm", optional: false]}], "hexpm", "7d98bac1ee4565d31b62d59f8823dfd8356a169e7fcbb83831b8a5397404c9de"}, "cowlib": {:hex, :cowlib, "2.12.1", "a9fa9a625f1d2025fe6b462cb865881329b5caff8f1854d1cbc9f9533f00e1e1", [:make, :rebar3], [], "hexpm", "163b73f6367a7341b33c794c4e88e7dbfe6498ac42dcd69ef44c5bc5507c8db0"}, + "credo": {:hex, :credo, "1.7.1", "6e26bbcc9e22eefbff7e43188e69924e78818e2fe6282487d0703652bc20fd62", [:mix], [{:bunt, "~> 0.2.1", [hex: :bunt, repo: "hexpm", optional: false]}, {:file_system, "~> 0.2.8", [hex: :file_system, repo: "hexpm", optional: false]}, {:jason, "~> 1.0", [hex: :jason, repo: "hexpm", optional: false]}], "hexpm", "e9871c6095a4c0381c89b6aa98bc6260a8ba6addccf7f6a53da8849c748a58a2"}, "db_connection": {:hex, :db_connection, "2.6.0", "77d835c472b5b67fc4f29556dee74bf511bbafecdcaf98c27d27fa5918152086", [:mix], [{:telemetry, "~> 0.4 or ~> 1.0", [hex: :telemetry, repo: "hexpm", optional: false]}], "hexpm", "c2f992d15725e721ec7fbc1189d4ecdb8afef76648c746a8e1cad35e3b8a35f3"}, "decimal": {:hex, :decimal, "2.1.1", "5611dca5d4b2c3dd497dec8f68751f1f1a54755e8ed2a966c2633cf885973ad6", [:mix], [], "hexpm", "53cfe5f497ed0e7771ae1a475575603d77425099ba5faef9394932b35020ffcc"}, "dns_cluster": {:hex, :dns_cluster, "0.1.1", "73b4b2c3ec692f8a64276c43f8c929733a9ab9ac48c34e4c0b3d9d1b5cd69155", [:mix], [], "hexpm", "03a3f6ff16dcbb53e219b99c7af6aab29eb6b88acf80164b4bd76ac18dc890b3"}, diff --git a/test/support/data_case.ex b/test/support/data_case.ex index e9c735d1..38036b68 100644 --- a/test/support/data_case.ex +++ b/test/support/data_case.ex @@ -15,6 +15,7 @@ defmodule Radiator.DataCase do """ use ExUnit.CaseTemplate + alias Ecto.Adapters.SQL.Sandbox using do quote do @@ -36,8 +37,8 @@ defmodule Radiator.DataCase do Sets up the sandbox based on the test tags. """ def setup_sandbox(tags) do - pid = Ecto.Adapters.SQL.Sandbox.start_owner!(Radiator.Repo, shared: not tags[:async]) - on_exit(fn -> Ecto.Adapters.SQL.Sandbox.stop_owner(pid) end) + pid = Sandbox.start_owner!(Radiator.Repo, shared: not tags[:async]) + on_exit(fn -> Sandbox.stop_owner(pid) end) end @doc """