Skip to content

Conversation

@dantswain
Copy link
Collaborator

Just bringing these libs up to date. No actual code change. I haven't overridden anything with the .credo.exs, but it seems to be
good practice to have it in place.

I haven't overridden anything with the .credo.exs, but it seems to be
good practice to have it in place.
@dantswain dantswain requested review from bjhaid and joshuawscott July 25, 2017 20:46
{Credo.Check.Design.DuplicatedCode, excluded_macros: []},

# You can also customize the exit_status of each check.
# If you don't want TODO comments to cause `mix credo` to fail, just

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO found

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤦

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👏 @ebert[bot]

@sourcelevel-bot
Copy link

Ebert has finished reviewing this Pull Request and has found:

  • 1 possible new issue (including those that may have been commented here).
  • 36 fixed issues! 🎉

You can see more details about this review at https://ebertapp.io/github/kafkaex/kafka_ex/pulls/218.

Copy link
Member

@joshuawscott joshuawscott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@dantswain dantswain closed this Jul 26, 2017
@dantswain dantswain deleted the update_static_analysis branch July 26, 2017 14:54
@dantswain dantswain restored the update_static_analysis branch July 26, 2017 14:54
@dantswain dantswain reopened this Jul 26, 2017
@dantswain dantswain merged commit 8666826 into master Jul 26, 2017
@dantswain dantswain deleted the update_static_analysis branch July 26, 2017 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants