-
Notifications
You must be signed in to change notification settings - Fork 296
Conversation
Finally figured out how to structure travis config and the results look good. One inttest CI timeout on 18 and as expected errors in the separate Dialyzer job. If we don't merge this, we will only accumulate more Dialyzer errors. |
Waiting for #643 to be merged. |
With the two silenced opaque match warnings, we're down to this (from https://travis-ci.org/rebar/rebar/jobs/266578662):
I want to say when calling rebar_config:new, it should also set dir to CWD, but not sure.
Not sure. Maybe related to the rebar_config warning above?
Tried to fix this, but couldn't figure it out. |
@lrascao, maybe Dialyzer is more capable in 20.0 and has revealed issues that are quite old. I mean, that discrepancy in |
The only idea i have is to use cwd, but as you say, it's probably not worth the risk, if there's a way to instead silence the warning that would be preferable |
In order to get this merged and not get lost, we now allow the Dialyzer job to fail. |
There are new Dialyzer errors. To not introduce any new warnings, try to enable Dialyzer on Travis-CI.