[PATCH] D116638: [clang-format] Fix ignoring JavaScriptWrapImport when ColumnWidth: 0

[MyDeveloperDay via Phabricator via cfe-commits]

MyDeveloperDay added a comment.

> Make JavaScriptWrapImports: true *always* wrap imports to multiple lines. 
> This will be noisy and ugly.

Isn't this what `prettier` does when effectively the ColumnLimit is exceeded.

i.e.

  import { Controller, Get, Post, Req } from '@nestjs/common';

becomes as I hit the 80 column mark

  `
  import {
    Controller,
    Get,
    Post,
    Req,
    Request,
    Param,
    Query,
    StreamableFile,
    Body,
  } from '@nestjs/common';

So if ColumnLimit is 0 it should wrap them shouldn't it if 
`JavaScriptWrapImports: true`


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116638/new/

https://reviews.llvm.org/D116638

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits