[PATCH] D118361: clang-format: [JS] sort import aliases.

2022-01-27 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka added a comment. reverted with f750c3d95a0c8bf1d21380ae753fce12010a7561 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118361/new/ https://reviews.llvm.org/D118361

[PATCH] D118361: clang-format: [JS] sort import aliases.

2022-01-27 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka added a comment. After this patch https://lab.llvm.org/buildbot/#/builders/74/builds/9049/steps/15/logs/stdio Note: Google Test filter = SortImportsTestJS.ImportEqAliases [==] Running 1 test from 1 test suite. [--] Global test environment set-up. [--]

[PATCH] D118361: clang-format: [JS] sort import aliases.

2022-01-27 Thread Martin Probst via Phabricator via cfe-commits
mprobst updated this revision to Diff 403646. mprobst added a comment. - make test break if we used alphasort Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118361/new/ https://reviews.llvm.org/D118361 Files:

[PATCH] D118361: clang-format: [JS] sort import aliases.

2022-01-27 Thread Martin Probst via Phabricator via cfe-commits
mprobst abandoned this revision. mprobst added a comment. Superseded by https://reviews.llvm.org/D118363 (sorry for the diff confusion). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118361/new/ https://reviews.llvm.org/D118361

[PATCH] D118361: clang-format: [JS] sort import aliases.

2022-01-27 Thread Martin Probst via Phabricator via cfe-commits
mprobst created this revision. mprobst added a reviewer: krasimir. Herald added a subscriber: jeroen.dobbelaere. mprobst requested review of this revision. Herald added a project: clang. Users can define aliases for long symbols using import aliases: import X = A.B.C; Previously, these were