- User Since
- Sep 13 2020, 3:46 AM (30 w, 3 d)
Mar 9 2021
Mar 2 2021
Thank you for delivering it, @HazardyKnusperkeks.
Feb 28 2021
Feb 27 2021
Do you need someone to push it?
Yes, I need someone to do that. I don't have write permissions. I'm new here.
Feb 12 2021
Rebased to the tip of the main branch, merged similar tests (requested by @curdeius)
Feb 10 2021
Rebased to the tip of the main branch, sorted code alphabetically as suggested by @HazardyKnusperkeks and applied almost all fixes proposed by @curdeius (entry in release notes + some changes in unit tests)
Dec 21 2020
Renamed MaxUnwrappedLinesForShortNamespace to ShortNamespaceLines (thank you @MyDeveloperDay for the idea and @curdeius for mentioning clang-tidy). I haven't changed the description of this option so you might wanna complain on that. Tbh. English isn't my native language so I'll take your tips if you wanna help simplify it.
Dec 19 2020
As proposed by @MyDeveloperDay, I added MaxUnwrappedLinesForShortNamespace option to control behaviour of FixNamespaceComments. This time I'm providing you with a full diff.
Btw. sorry for taking so long.
Sep 16 2020
Thanks @MyDeveloperDay, I appreciate your input in this review. You're a good reviewer. :)
Also, I think you've made some good points there and I'm going to turn this review into "non-breaking the world" one. I'll be back in a few days with a new diff.
Sep 14 2020
Hey @MyDeveloperDay, thanks for the review.
Thanks @JakeMerdichAMD for your quick feedback.
Sep 13 2020
Formatted code after CI failure.
Adjusted tests IncludeFixerTest.cpp after CI failure.
This fixes https://bugs.llvm.org/show_bug.cgi?id=47290.