Added jsescOptions to Numeric Literals

[sidntrivedi012]

Q	A
Fixed Issues?	Fixes #10960
Patch: Bug Fix?
Major: Breaking Change?
Minor: New Feature?	Yes
Tests Added + Pass?	Yes
Documentation PR Link	babel/website#2172
Any Dependency Changes?
License	MIT

[sidntrivedi012]

Reffered the code from StringLiteral function for Numeric Literals. What can we use here instead of this.format.jsonCompatibleStrings for Numeric Literals?

[sidntrivedi012]

@JLHwung Made the changes. But it is affecting other tests. PTAL.

[sidntrivedi012]

@JLHwung @nicolo-ribaudo PTAL now.

• Have added conditions in types.js to check where jsesc will need to be applied.
• Have also added a test.
• Alongwith this, have made separate optional.json files for jsonEscape and NumericLiteral folders respectively in fixtures.

[nicolo-ribaudo]

I think that the logic should be something like this:

1. Is this.format.jsescOption.numbers is set,
   i. Call this.number(jsesc(node.value, this.format.jsescOption))
2. Otherwise, fallback to our old unmodified logic.

If someone sets this.format.jsescOption.numbers, they are explicitly overwriting any other automatic logic used to format numbers (i.e. literal length when minified: true, or "use the raw value if it's possible").

[sidntrivedi012]

@nicolo-ribaudo @JLHwung PTAL now. Have modified the conditions as mentioned above.

[sidntrivedi012]

@nicolo-ribaudo Made the changes. PTAL now.

[sidntrivedi012]

@nicolo-ribaudo Done. Please review.

[nicolo-ribaudo]

Thanks!