[x] 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.
I affirm:
Describe the feature
Looking at this PR: https://github.com/LandSandBoat/server/pull/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.