fix(metro-serializer-esbuild): default to the new hermes target

[tido64]

Description

Default to the new hermes target introduced in esbuild 0.14.48.

Resolves #1670.

Test plan

Existing tests should pass.

[ecreeth]

@tido64 I just found that in the esbuild compact table of features, the ConstAndLet one is added in the hermes target. I don't know how they handle that, but i think is wrong.

[tido64]

@tido64 I just found that in the esbuild compact table of features, the ConstAndLet one is added in the hermes target. I don't know how they handle that, but i think is wrong.

@ecreeth: No, I think this is "correct". Hermes does support const, kind of. It supports the keyword, but none of its features 😛 Not sure if it affects how esbuild transpiles stuff. Do you know? Did you file an issue?

[ecreeth]

@ecreeth: No, I think this is "correct". Hermes does support const, kind of. It supports the keyword, but none of its features 😛 Not sure if it affects how esbuild transpiles stuff. Do you know? Did you file an issue?

🤷‍♂️ I don't know how they do it, nor did I fill out a issue.

[tido64]

Filed evanw/esbuild#2365. Let's see what Evan says.

@ecreeth: Thanks again for being on top of these things. I really appreciate it. 🙏