dwyl / dwyl/library

Lotsa Warnings ... ⚠️> Update Phoenix/Elixir Dependencies

Open
#85 2 comments 1 reaction 0 assignees View on GitHub
chore help wanted priority-2
Dominant language
Elixir
Stars
16
Forks
1
PR merge metrics
No merged PRs in 30d

Description

When running (compiling) the project on `localhost` using `Elixir 1.8.1` we see _many_ warnings ...
![image](https://user-images.githubusercontent.com/194400/60166464-27afe300-97f9-11e9-9741-4292b1d039c0.png)

_Most_ (_if not all_) of these are due to the underlying libraries `Phoenix`, `Ecto`, `Poison`, etc. being out-of-date. So we need to run [`mix deps.update`](https://tosbourn.com/keeping-elixir-packages-updated/) and fix any broken tests.

# Todo
+ [ ] Update all Elixir/Phoenix packages to latest versions
+ [ ] Fix any broken tests
+ [ ] ideally we should have ` --warnings-as-errors` enabled in a project that is kept up-to-date.
That way _anyone_ working on the project will _immediately_ see when a warning appears.
It was removed in https://github.com/dwyl/library/commit/f38b2c4e43d094943182be4a89c72406994019d0 to make Travis-CI pass, but I feel it should be re-instated.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.