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

Add Math.random #190

Merged
merged 1 commit into from
Feb 17, 2020
Merged

Add Math.random #190

merged 1 commit into from
Feb 17, 2020

Conversation

hovsater
Copy link
Contributor

@hovsater hovsater commented Feb 16, 2020

I was not sure if we were missing Math.random or if there is valid reason to why we're not supporting it yet. Feel free to close this pull request with an explanation if that's the case. 🙂

@Namek
Copy link
Contributor

Namek commented Feb 16, 2020

Personally, I would be after a Random module that would support deterministic results based on seeds. Of course, we need also the Math.random() to somehow produce a randomized seed but maybe better in the other module rather than Math. I would love to hear your inputs!

@gdotdesign
Copy link
Member

I think Math.random have it's use cases and there is no reason why there cannot be a Random module that supports a seed.

Thanks for the PR 👍

@gdotdesign gdotdesign merged commit ffe876b into mint-lang:master Feb 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants