[PATCH] D152589: [clang-tidy] Add readability test for not allowing relative includes

2023-07-21 Thread Erez Amihud via Phabricator via cfe-commits
ErezAmihud abandoned this revision. ErezAmihud added a comment. Closed, because I don't have time to work on it Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D152589/new/ https://reviews.llvm.org/D152589

[PATCH] D152589: Add readability test for not allowing relative includes

2023-06-09 Thread Erez Amihud via Phabricator via cfe-commits
ErezAmihud created this revision. ErezAmihud added reviewers: clang-tools-extra, njames93. ErezAmihud created this object with edit policy "Administrators". ErezAmihud added a project: clang-tools-extra. Herald added subscribers: PiotrZSL, carlosgalvezp. Herald added a project: All. ErezAmihud requ

[PATCH] D152589: Add readability test for not allowing relative includes

2023-06-09 Thread Erez Amihud via Phabricator via cfe-commits
ErezAmihud updated this revision to Diff 530102. ErezAmihud added a comment. Remove a line in absolute-includes-only.cpp (fix formatting) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D152589/new/ https://reviews.llvm.org/D152589 Files: clang-tools-extra/clang-tidy/readability/Absolut

[PATCH] D152589: [clang-tidy] Add readability test for not allowing relative includes

2023-06-13 Thread Erez Amihud via Phabricator via cfe-commits
ErezAmihud added a comment. In D152589#4410930 , @PiotrZSL wrote: > Even with that for me for example, I would consider `#include "xyz"` be fine, > but `#include "../xyz"` would be a red flag, so please consider adding > `StrictMode` option to check so