go-exploit icon indicating copy to clipboard operation
go-exploit copied to clipboard

Remove `random` package `Random` prefixes.

Open terrorbyte opened this issue 8 months ago • 2 comments

Having every function prefixed with Random is redundant and can sometimes cause more confusion than it helps. This is a pretty big change and should not be merged until supported packages can change.

terrorbyte avatar Mar 20 '25 21:03 terrorbyte

This would break the API. Unless you are proposing a major version bump, which I'd be hard pressed to agree with, this is sort of DOA. While I agree that the naming scheme is imperfect, it's certainly not worthy of breaking others integrations nor introducing a major version update.

j-baines avatar Mar 24 '25 12:03 j-baines

Fair and reasonable. I considered deprecating the old functions but leaving them to keep stability. I'll make a issue and maybe add a "api-break" change comment leading here.

terrorbyte avatar Mar 24 '25 20:03 terrorbyte