[PATCH] D112572: [docs][clang-format] warn on \code block indentation error

2021-10-30 Thread MyDeveloperDay via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG931d20c5db05: [docs][clang-format] warn on \code block indentation error (authored by FederAndInk, committed by MyDeveloperDay). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://

[PATCH] D112572: [docs][clang-format] warn on \code block indentation error

2021-10-29 Thread Björn Schäpers via Phabricator via cfe-commits
HazardyKnusperkeks added a comment. In D112572#3089373 , @FederAndInk wrote: > In D112572#3089318 , > @HazardyKnusperkeks wrote: > >> Thanks for fixing my mistake. :) > > You're welcome ;) > > Thanks for your re

[PATCH] D112572: [docs][clang-format] warn on \code block indentation error

2021-10-26 Thread Ludovic Jozeau via Phabricator via cfe-commits
FederAndInk added a comment. In D112572#3089318 , @HazardyKnusperkeks wrote: > Thanks for fixing my mistake. :) You're welcome ;) Thanks for your reviews. Should we add other reviewers? Also, I don't have commit rights, can one of you commit it on my

[PATCH] D112572: [docs][clang-format] warn on \code block indentation error

2021-10-26 Thread Björn Schäpers via Phabricator via cfe-commits
HazardyKnusperkeks added a comment. Thanks for fixing my mistake. :) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112572/new/ https://reviews.llvm.org/D112572 ___ cfe-commits mailing list cfe-commits@li

[PATCH] D112572: [docs][clang-format] warn on \code block indentation error

2021-10-26 Thread Ludovic Jozeau via Phabricator via cfe-commits
FederAndInk added a comment. In D112572#3088800 , @MyDeveloperDay wrote: > It seems like you are fixing more than just the bug you highlight, i.e looks > you might be fixing some autopep8 or pylint issues too. Yes, I also fixed some minor issues, my ba

[PATCH] D112572: [docs][clang-format] warn on \code block indentation error

2021-10-26 Thread MyDeveloperDay via Phabricator via cfe-commits
MyDeveloperDay accepted this revision. MyDeveloperDay added a comment. This revision is now accepted and ready to land. It seems like you are fixing more than just the bug you highlight, i.e looks you might be fixing some autopep8 or pylint issues too. As this isn't a tool we ship, I don't have