MyDeveloperDay added a comment.
Thank you for the patch, could you just handle the documentation, but otherwise
pretty good
================
Comment at: clang/docs/ClangFormatStyleOptions.rst:2820
+**JavaScriptWrapImports** (``Boolean``) :versionbadge:`clang-format 3.9`
+ Whether to wrap JavaScript import/export statements. If ``false``, then
----------------
This file is generated from Format.h you need to make the changes there and
regenerate using clang/docs/tools/dump_format_style.py
================
Comment at: clang/unittests/Format/FormatTestJS.cpp:1948
+ Style.JavaScriptWrapImports = false;
verifyFormat("import {VeryLongImportsAreAnnoying,
VeryLongImportsAreAnnoying,"
" VeryLongImportsAreAnnoying, VeryLongImportsAreAnnoying"
----------------
you are no longer testing the LLVM Case, please don't remove that, but feel
free to ensure they are doing the same for google!
Repository:
rG LLVM Github Monorepo
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
