[PATCH] D146759: Declaration fragments for properties include additional property attributes that were not specified in the sources proposed solution

[Zixu Wang via Phabricator via llvm-commits]

zixuw added a comment.

In D146759#4225656 <https://reviews.llvm.org/D146759#4225656>, @Unique_Usman wrote:

> Thanks for your reply sir. Regarding the comment, I would remove it sir.
>
> Sir, about updating/adding the test cases, I have never written any test case before sir, kindly guide me through that sir?

I cannot hold your hands through everything. You can learn a lot by looking at other people's commits and previous commits. Go ahead and find some simple commits, and check out how other people are writing a patch with a test case, and how other people write the commit message.
ExtractAPI tests are located under `clang/test/ExtractAPI/`.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D146759/new/

https://reviews.llvm.org/D146759