-
-
Notifications
You must be signed in to change notification settings - Fork 671
Implement new concept with exercise: randomness #2139
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
Conversation
Dear norbs57Thank you for contributing to the Go track on Exercism! 💙
Dear Reviewer/Maintainer
Automated comment created by PR Commenter 🤖. |
47c4c58 to
96fe457
Compare
Fixes exercism#2038 Implements concept randomness together with a concept exercise. The exercise was partly ported from the roll-a-die exercise on the C# track. Updated root-level config.json
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working on this. I really like the introduction to the subject you did, is short, straight to the point and the examples are relevant.
Left some minor comments.
Co-authored-by: André Santos <[email protected]>
Co-authored-by: André Santos <[email protected]>
Co-authored-by: André Santos <[email protected]>
Co-authored-by: André Santos <[email protected]>
Co-authored-by: André Santos <[email protected]>
Co-authored-by: André Santos <[email protected]>
Co-authored-by: André Santos <[email protected]>
|
You may have seen it on https://pkg.go.dev/math/rand
…On Mon, 28 Mar 2022 at 23:56, André Santos ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In concepts/randomness/about.md
<#2139 (comment)>:
> +### Caution
+
+The output produced by package `math/rand` might be easily predictable regardless of how it is seeded.
+For random numbers suitable for security-sensitive work, you should use the [`crypto/rand`][cryptorand] package.
You are right, thought I've seen this somewhere else, disregard the advice
then.
—
Reply to this email directly, view it on GitHub
<#2139 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AG2445JX6JTLYWYUSJGRF3LVCI2IJANCNFSM5RVL5LZA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Fixes #2038
Implements concept randomness including an exercise
The exercise was partly ported from the roll-a-die exercise on the C# track