Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(Compiler Warning): Change from crypto:rand_uniform/2 to rand:un… #172

Merged
merged 1 commit into from
Jul 10, 2019

Conversation

tomciopp
Copy link
Contributor

…iform/1.

Splitting code changes from #156 into two PRs based on suggestion by @arjan.

@seriyps
Copy link
Collaborator

seriyps commented Jun 3, 2019

Module rand was added in OTP 18, but in another PR you have platform_define even for R16.
But I think it should be ok to support only OTP >= 18, just .travis.yml should be updated (however, travis is not enabled now)

@seriyps seriyps mentioned this pull request Jun 5, 2019
@arjan
Copy link
Collaborator

arjan commented Jul 10, 2019

Merging since we now require at least OTP 18

@arjan arjan merged commit 3d806f8 into gen-smtp:master Jul 10, 2019
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