$ now works for unsigned intergers with nim js

[timotheecour]

No description provided.

[Araq]

This patch would be better without introducing your own ideas about how semver should work.

[timotheecour]

PTAL

This patch would be better without introducing your own ideas about how semver should work.

fine, I've reverted NimPatch bump from this PR, that aspect can be discussed separately in #14124

[narimiran]

All changes in system.nim could be (and should be, IMO) reverted. They have nothing to do with the stuff this PR tries to fix.

EDIT: Ok, dollars need version info, but no need to move it 2000 lines above.

[timotheecour]

All changes in system.nim could be (and should be, IMO) reverted. They have nothing to do with the stuff this PR tries to fix.

"all changes" is litterally only moving the nim version tuple, and is a required change; the reason I had moved it to the top is that it's a commonly used symbol for versioning, and it makes sense to define it as early as possible, instead of having to move it each time. It already has to be separated from NimVersion anyways, so 2000 lines vs 4 lines makes only a cosmetic difference.

but I've now moved it right above system/dollars not far from Nimversion for the sake of moving on.