You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I have searched existing issues to see if the issue has already been opened, and I have checked the commit log to see if the issue has been resolved since my server was last updated.
I see we have 100 results for math.random() with no arguments, which returns a float between 0-1, which we typically are multiplying by 100 and then flooring.
We have 4 results for the regex: random\(\d\).
I think we should be using math.random(1, 100) in pretty much every case, unless we're picking from a container, matching onto stats, etc.
The text was updated successfully, but these errors were encountered:
I affirm:
Describe the feature
Looking at this PR: #6406
I see we have 100 results for
math.random()
with no arguments, which returns a float between 0-1, which we typically are multiplying by 100 and then flooring.We have 4 results for the regex:
random\(\d\)
.I think we should be using
math.random(1, 100)
in pretty much every case, unless we're picking from a container, matching onto stats, etc.The text was updated successfully, but these errors were encountered: