<div dir="ltr">That's what happens when you hit the column limit, when there is a column limit. But do we really want every one-symbol import to wrap to 3 lines when `ColumnLimit: 0`? Slash to force the user to unwrap every import, even 20-symbol 300-column imports, to a single line?</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jan 5, 2022 at 12:13 PM MyDeveloperDay via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">MyDeveloperDay added a comment.<br>
<br>
> Make JavaScriptWrapImports: true *always* wrap imports to multiple lines. This will be noisy and ugly.<br>
<br>
Isn't this what `prettier` does when effectively the ColumnLimit is exceeded.<br>
<br>
i.e.<br>
<br>
  import { Controller, Get, Post, Req } from '@nestjs/common';<br>
<br>
becomes as I hit the 80 column mark<br>
<br>
  `<br>
  import {<br>
    Controller,<br>
    Get,<br>
    Post,<br>
    Req,<br>
    Request,<br>
    Param,<br>
    Query,<br>
    StreamableFile,<br>
    Body,<br>
  } from '@nestjs/common';<br>
<br>
So if ColumnLimit is 0 it should wrap them shouldn't it if `JavaScriptWrapImports: true`<br>
<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D116638/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D116638/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D116638" rel="noreferrer" target="_blank">https://reviews.llvm.org/D116638</a><br>
<br>
</blockquote></div>