Eugene.Zelenko added a comment. Please rebase from trunk.
================ Comment at: docs/ReleaseNotes.rst:63 + + Checks to ensure code does not open `namespace absl` as that + violates Abseil's compatibility guidelines. ---------------- Just Ensures. Please use `` for language constructs. ================ Comment at: docs/clang-tidy/checks/abseil-no-namespace.rst:6 + +This check gives users a warning if they attempt to open ``namespace absl``. +Users should not open ``namespace absl`` as that conflicts with abseil's ---------------- Please synchronize first sentence with Release Notes. ================ Comment at: docs/clang-tidy/checks/abseil-no-namespace.rst:7 +This check gives users a warning if they attempt to open ``namespace absl``. +Users should not open ``namespace absl`` as that conflicts with abseil's +compatibility guidelines and may result in breakage. ---------------- Abseil ================ Comment at: docs/clang-tidy/checks/abseil-no-namespace.rst:10 + +Any user that uses: + ---------------- Code instead of user, please. ================ Comment at: docs/clang-tidy/checks/abseil-no-namespace.rst:20 + +See `the full Abseil compatibility guidelines <https: // +abseil.io/about/compatibility>`_ for more information. ---------------- Please remove space after https: https://reviews.llvm.org/D50580 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits