fix warnings after Elixir version 1.3 and add `mix.lock` and `.gitignore`

ma2gedev / power_assert_ex

Power Assert in Elixir. Shows evaluation results each expression.

https://github.com/ma2gedev/power_assert_ex

Other

214 stars 6 forks source link

fix warnings after Elixir version 1.3 and add `mix.lock` and `.gitignore` #10

Closed melpon closed 7 years ago

melpon commented 7 years ago

Elixir 1.4.2 で、現在 issue に上がっている #8 を除いて mix test が通ることを確認しています。

警告が出ていたので直しました
git status 等を実行した時に _build/ や deps/ が変更に出てきて邪魔だったので、mix new で自動生成したプロジェクトに入っている .gitignore をコピーしておきました。
mix.lock ファイルが無かったので追加しておきました。mix.lock ファイルは、依存しているライブラリのバージョンを固定するために必要なので、付けておいた方が良いと思います。
test/ の警告は直していません

ma2gedev commented 7 years ago

Hello, @melpon! This is your first Pull Request that will be reviewed by Ebert, an automatic Code Review service. It will leave comments on this diff with potential issues and style violations found in the code as you push new commits. You can also see all the issues found on this Pull Request on its review page. Please check our documentation for more information.

ma2gedev commented 7 years ago

Ebert has finished reviewing this Pull Request and has found:

3 fixed issues! 🎉

You can see more details about this review at https://ebertapp.io/github/ma2gedev/power_assert_ex/pulls/10.

melpon commented 7 years ago

あ、既に https://github.com/ma2gedev/power_assert_ex/tree/fix-warnings-after-1.3 で直していたのに気が付きませんでした。 .gitignore と mix.lock 追加するのだけ出し直します！

ma2gedev commented 7 years ago

@melpon Thank you for your pull request 😄

Thanks for fixing warnings, adding .gitignore and mix.lock that are helpful. So I will re-open this pull request and merge.

Please do not mind my branch fix-warnings-after-1.3, I am worried because I did it while I was sleepy. 😱

melpon commented 7 years ago

Thank you for merging my pull request!

ma2gedev commented 7 years ago

Just published v0.1.0 to hex! https://hex.pm/packages/power_assert