React.lazy with dynamic import should be printed in the same way

Prettier 1.14.3 Playground link

--parser babylon
--print-width 50

Input:

import Home from './Home'
import SomeLongLongComponent from './SomeLongLongComponent'

const Home = React.lazy(() => import('./Home'))
const SomeLongLongComponent = React.lazy(() => import('./SomeLongLongComponent'))

Output:

import Home from "./Home";
import SomeLongLongComponent from "./SomeLongLongComponent";

const Home = React.lazy(() => import("./Home"));
const SomeLongLongComponent = React.lazy(() =>
  import("./SomeLongLongComponent")
);

Expected behavior:

import Home from "./Home";
import SomeLongLongComponent from "./SomeLongLongComponent";

const Home = React.lazy(() => import("./Home"));
const SomeLongLongComponent = React.lazy(() => import("./SomeLongLongComponent"));

or

import Home from "./Home";
import SomeLongLongComponent from "./SomeLongLongComponent";

const Home = React.lazy(() =>
  import("./Home")
);
const SomeLongLongComponent = React.lazy(() =>
  import("./SomeLongLongComponent")
);

[duailibe]

I can’t see the need for this.

I don’t think we should special case React.lazy at all, but we could change the formatting of a lambda if the body is an import() expression. I don’t see why though…

[hawkrives]

Yeah, agreed - it is a bit of a stretch.

I haven’t done a lot of work with dynamic imports yet, so I don’t have a very strong opinion at this time.