On Thu, Nov 29, 2018 at 12:20 PM Boris Zbarsky <bzbar...@mit.edu> wrote:
> On 11/29/18 11:40 AM, Ehsan Akhgari wrote: > > Yes, that is true sadly. But to be fair here, old mq patches that do not > > apply due to age are already beyond saving with any kind of automated > > tooling, and they require manual work to get them applied first. :-/ > > Sure. > > > That's not true. clang-format can happily format diffs. When formatting > > diffs though it tries to format your changes, not the context around the > > diffs > > Ah, OK. That makes sense. > > > * Update your working tree to the parent of the commit that did the > > reformat. > > This also makes sense. > > Have we considered adding an easy-to-remember tag for that commit to > make that easier in the future? > Yes, please check out https://bugzilla.mozilla.org/show_bug.cgi?id=1508324. (For git users, its sha1 will be recorded in .git-blame-ignore-revs for the benefit of the git hyper-blame command[1].) [1] https://commondatastorage.googleapis.com/chrome-infra-docs/flat/depot_tools/docs/html/git-hyper-blame.html -- Ehsan _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
