stephanemoore marked an inline comment as done. stephanemoore added inline comments.
================ Comment at: docs/ReleaseNotes.rst:248 +- The `Acronyms` and `IncludeDefaultAcronyms` options for the + :doc:`objc-property-declaration<clang-tidy/checks/objc-property-declaration>` ---------------- stephanemoore wrote: > Eugene.Zelenko wrote: > > Please rebase from trunk and use :option: prefix for options.. > Could using the :option: prefix cause issues if I am removing the option? > > The documentation generation seems to emit a warning for an unknown option: > "llvm/src/tools/clang/tools/extra/docs/ReleaseNotes.rst:76: WARNING: unknown > option: Acronyms" > http://lab.llvm.org:8011/builders/clang-tools-sphinx-docs/builds/36425/steps/docs-clang-tools-html/logs/stdio FYI: I rolled this change back due to the documentation generation failure and sent out https://reviews.llvm.org/D57080 with amended changes. Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56945/new/ https://reviews.llvm.org/D56945 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
