Skip to content

OTP 19 dialyzer + OTP 21 compilation issues#105

Closed
paulo-ferraz-oliveira wants to merge 2 commits intodevinus:masterfrom
paulo-ferraz-oliveira:fix/otp_19_dialyzer_otp_21_compil
Closed

OTP 19 dialyzer + OTP 21 compilation issues#105
paulo-ferraz-oliveira wants to merge 2 commits intodevinus:masterfrom
paulo-ferraz-oliveira:fix/otp_19_dialyzer_otp_21_compil

Conversation

@paulo-ferraz-oliveira
Copy link

No description provided.

@getong
Copy link
Contributor

getong commented Jul 11, 2018

@paulo-ferraz-oliveira I think my pr #105 is better than yours, take a look at it.

@paulo-ferraz-oliveira
Copy link
Author

@getong, do you mean #104? [better is a relative term, I guess :D]

In any case, you didn't include:

I'll close it since it seems nobody's paying attention to PRs anyway :D (the last change was 2 years ago and there's about 15 unanswered PRs).

@Vagabond
Copy link
Collaborator

If you want to add the extra versions to Travis for this PR and re-open it, I can merge it.

@Vagabond
Copy link
Collaborator

Actually, I'm not sure on the OTP compatability guidelines for poolboy, but surely we can drop R16 at this point?

@getong
Copy link
Contributor

getong commented Jul 12, 2018

@Vagabond #104 test erlang version from 17 to 21, and all tests passed, I think you can merge that first. And a new tag need to be pushed to hex.pm.

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.

3 participants