RFC: <template> in CallExpression argument position

[gitKrystan]

Vote by reacting to this issue. Comment with more details below. Do you prefer:

Option A (🚀)

This is what v2.0.0-0 is currently returning.

await render(
  <template>
    <h1>I am a multi-line template</h1>
    <p>With some text</p>
  </template>,
);

Option B (👎)

This is what v1 currently returns.

await render(<template>
  <h1>I am a multi-line template</h1>
  <p>With some text</p>
</template>);

[NullVoxPopuli]

I always format my own stuff as the option a anyway. I think it looks better

[IgnaceMaes]

Is that trailing comma in option A intentional?

I personally also format my tests as described in option A. One downside is that it has an extra level of indentation over option B. But considering test templates usually are quite flat, it's not really an issue.

[gitKrystan]

Is that trailing comma in option A intentional?

Yes, it comes from prettier's default trailing comma config. You can change the config (setting it to 'es5' should do it) but it would affect all your JS, not just the JS with <template> arguments. I kinda hate the comma too since we know render only takes one argument, but I'm not sure of a good way to remove it. Maybe special-casing render.