Add Credo to Project #46

Merged
merged 1 commit into from Jan 11, 2017

Projects

None yet

2 participants

@ybur-yug
Contributor

By request from @doomspork.

This simply adds it to the project and deals with any nags that came from running it (not in strict mode, though)

lib/ueberauth/strategies/helpers.ex
@@ -76,9 +76,10 @@ defmodule Ueberauth.Strategy.Helpers do
"""
@spec allowed_callback_method?(Plug.Conn.t) :: boolean
def allowed_callback_method?(%{method: method} = conn) do
+ callback_method = String.upcase(to_string(method))
@doomspork
doomspork Jan 11, 2017 Member
callback_method = 
  method
  |> to_string  
  |> String.upcase
@doomspork doomspork merged commit 79ba218 into ueberauth:master Jan 11, 2017
@doomspork
Member

Thank you for the quick turn around 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment