[PATCH] D56945: [clang-tidy] Delete obsolete objc-property-declaration options ✂️
Stephane Moore via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 25 15:59:58 PST 2019
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
More information about the cfe-commits
mailing list