[PATCH] D56945: [clang-tidy] Delete obsolete objc-property-declaration options ✂️

Stephane Moore via Phabricator via llvm-commits llvm-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 llvm-commits mailing list