Fix linting warnings

Author: VeljkoMaksimovic

ee/ephemeral_environments/lib/ephemeral_environments/grpc/ephemeral_environments_server.ex

@@ -1,36 +1,15 @@
 defmodule EphemeralEnvironments.Grpc.EphemeralEnvironmentsServer do
   use GRPC.Server, service: InternalApi.EphemeralEnvironments.EphemeralEnvironments.Service
 
-  alias InternalApi.EphemeralEnvironments.{
-    ListRequest,
-    ListResponse,
-    DescribeRequest,
-    DescribeResponse,
-    CreateRequest,
-    CreateResponse,
-    UpdateRequest,
-    UpdateResponse,
-    DeleteRequest,
-    DeleteResponse,
-    CordonRequest,
-    CordonResponse
-  }
+  alias EphemeralEnvironments.Service.EphemeralEnvironmentType
 
   def list(request, _stream) do
-    case EphemeralEnvironments.Service.EphemeralEnvironmentType.list(request.org_id) do
-      {:ok, environment_types} ->
-        %{environment_types: environment_types}
-
-      {:error, error_message} ->
-        raise GRPC.RPCError, status: :unknown, message: error_message
-    end
+    {:ok, environment_types} = EphemeralEnvironmentType.list(request.org_id)
+    %{environment_types: environment_types}
   end
 
   def describe(request, _stream) do
-    case EphemeralEnvironments.Service.EphemeralEnvironmentType.describe(
-           request.id,
-           request.org_id
-         ) do
+    case EphemeralEnvironmentType.describe(request.id, request.org_id) do
       {:ok, environment_type} ->
         # Note: instances field will be added once we implement instance management
         %{environment_type: environment_type, instances: []}
@@ -44,7 +23,7 @@ defmodule EphemeralEnvironments.Grpc.EphemeralEnvironmentsServer do
   end
 
   def create(request, _stream) do
-    case EphemeralEnvironments.Service.EphemeralEnvironmentType.create(request.environment_type) do
+    case EphemeralEnvironmentType.create(request.environment_type) do
       {:ok, ret} ->
         %{environment_type: ret}
 
@@ -54,7 +33,7 @@ defmodule EphemeralEnvironments.Grpc.EphemeralEnvironmentsServer do
   end
 
   def update(request, _stream) do
-    case EphemeralEnvironments.Service.EphemeralEnvironmentType.update(request.environment_type) do
+    case EphemeralEnvironmentType.update(request.environment_type) do
       {:ok, environment_type} ->
         %{environment_type: environment_type}
 
@@ -67,10 +46,8 @@ defmodule EphemeralEnvironments.Grpc.EphemeralEnvironmentsServer do
   end
 
   def delete(_request, _stream) do
-    %DeleteResponse{}
   end
 
   def cordon(_request, _stream) do
-    %CordonResponse{}
   end
 end

ee/ephemeral_environments/lib/ephemeral_environments/utils/proto.ex

@@ -54,8 +54,8 @@ defmodule EphemeralEnvironments.Utils.Proto do
   end
 
   def to_map(value), do: value
-  defp convert_value(value, module, field) when is_list(value), do: Enum.map(value, &to_map/1)
-  defp convert_value(value, module, field) when is_struct(value), do: to_map(value)
+  defp convert_value(value, _module, _field) when is_list(value), do: Enum.map(value, &to_map/1)
+  defp convert_value(value, _module, _field) when is_struct(value), do: to_map(value)
 
   defp convert_value(value, module, field) when is_integer(value) do
     # Check if this field is an enum by looking at the field definition

ee/ephemeral_environments/lib/internal_api/include/google/protobuf/timestamp.pb.ex

@@ -1,5 +1,4 @@
 list='
-include/google/protobuf/timestamp
 ephemeral_environments
 rbac
 '