Fix credo readability issues

Author: knewter

time_tracker_backend/lib/time_tracker_backend.ex

@@ -1,4 +1,9 @@
 defmodule TimeTrackerBackend do
+  @moduledoc """
+  This is primarily an example phoenix application for use showing off elm and
+  phoenix integration.
+  """
+
   use Application
 
   # See http://elixir-lang.org/docs/stable/elixir/Application.html

time_tracker_backend/lib/time_tracker_backend/guardian_serializer.ex

@@ -1,11 +1,12 @@
 defmodule TimeTrackerBackend.GuardianSerializer do
+  @moduledoc false
   @behaviour Guardian.Serializer
 
   alias TimeTrackerBackend.{Repo, User}
 
-  def for_token(user = %User{}), do: { :ok, "User:#{user.id}" }
-  def for_token(_), do: { :error, "Unknown resource type" }
+  def for_token(user = %User{}), do: {:ok, "User:#{user.id}"}
+  def for_token(_), do: {:error, "Unknown resource type"}
 
-  def from_token("User:" <> id), do: { :ok, Repo.get(User, id) }
-  def from_token(_), do: { :error, "Unknown resource type" }
+  def from_token("User:" <> id), do: {:ok, Repo.get(User, id)}
+  def from_token(_), do: {:error, "Unknown resource type"}
 end

time_tracker_backend/web/controllers/chart_controller.ex

@@ -3,15 +3,15 @@ defmodule TimeTrackerBackend.ChartController do
 
   def index(conn, _params) do
     conn
-    |> render("index.json", data: static_data())
+      |> render("index.json", data: static_data())
   end
 
   defp static_data() do
     [
-      [ 1448928000000, 2 ],
-      [ 1451606400000, 2 ],
-      [ 1454284800000, 1 ],
-      [ 1456790400000, 1 ]
+      [1_448_928_000_000, 2],
+      [1_451_606_400_000, 2],
+      [1_454_284_800_000, 1],
+      [1_456_790_400_000, 1]
     ]
   end
 end

time_tracker_backend/web/models/organization.ex

@@ -1,4 +1,8 @@
 defmodule TimeTrackerBackend.Organization do
+  @moduledoc """
+  Organizations have many users and projects
+  """
+
   use TimeTrackerBackend.Web, :model
 
   schema "organizations" do

time_tracker_backend/web/models/project.ex

@@ -1,4 +1,8 @@
 defmodule TimeTrackerBackend.Project do
+  @moduledoc """
+  Projects in our system
+  """
+
   use TimeTrackerBackend.Web, :model
 
   schema "projects" do

time_tracker_backend/web/models/user.ex

@@ -1,4 +1,8 @@
 defmodule TimeTrackerBackend.User do
+  @moduledoc """
+  Users of our system.
+  """
+
   use TimeTrackerBackend.Web, :model
 
   schema "users" do

time_tracker_backend/web/resolvers/user_resolver.ex

@@ -1,4 +1,8 @@
 defmodule TimeTrackerBackend.UserResolver do
+  @moduledoc """
+  Functions used to resolve user-related data for our GraphQL endpoint
+  """
+
   alias TimeTrackerBackend.{Repo, User}
 
   def all(_args, _info) do

time_tracker_backend/web/schema.ex

@@ -1,4 +1,5 @@
 defmodule TimeTrackerBackend.Schema do
+  @moduledoc "The schema for our GraphQL endpoint"
   use Absinthe.Schema
   import_types TimeTrackerBackend.Schema.Types
 

time_tracker_backend/web/schema/types.ex

@@ -1,4 +1,5 @@
 defmodule TimeTrackerBackend.Schema.Types do
+  @moduledoc "The types used in our GraphQL endpoint"
   use Absinthe.Schema.Notation
 
   @desc "A user of our system"

time_tracker_backend/web/web.ex

@@ -39,12 +39,12 @@ defmodule TimeTrackerBackend.Web do
 
       alias TimeTrackerBackend.{User, Project, Organization}
 
-      defp order(query, "asc "<>field) do
+      defp order(query, "asc " <> field) do
         field = String.to_existing_atom(field)
         from u in query,
           order_by: [asc: ^field]
       end
-      defp order(query, "desc "<>field) do
+      defp order(query, "desc " <> field) do
         field = String.to_existing_atom(field)
         from u in query,
           order_by: [desc: ^field]