-
Andreas Haas authored
This reverts commit 42a51f6a. Reason for revert: With this change also JavaScript code in the V8 code base gets formatted. However, clang-format cannot deal with V8 intrinsics, and therefore inserts a space between the `%` and the intrinsics name. With that Clusterfuzz cannot detect intrinsics anymore, and therefore the blacklisting of intrinsics would not work anymore. The result is that Clusterfuzz then produces useless test cases that are known to crash and could not exist in the wild. Original change's description: > Enable Javascript formatting by default > > This removes the need for explicitly passing `--js` to `git cl format` > to format both JavaScript and TypeScript. It would reduce a significant > amount of mental overhead for formatting code in DevTools frontend, > which is primarily JS/TS. > > Fixed: 1007347 > Change-Id: Ia35c9af677d12ce9be72fe5fcdd6ceb6cb039c5f > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2029371 > Reviewed-by: Anthony Polito <apolito@google.com> > Commit-Queue: Tim van der Lippe <tvanderlippe@chromium.org> TBR=tvanderlippe@chromium.org,apolito@google.com,infra-scoped@luci-project-accounts.iam.gserviceaccount.com # Not skipping CQ checks because original CL landed > 1 day ago. Change-Id: I13e8ce8916f0685b667607f2c8e68b1185d43329 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2041650Reviewed-by: Tamer Tas <tmrts@chromium.org> Commit-Queue: Andreas Haas <ahaas@chromium.org>
417d89c3