vbvictor wrote: > > LGTM, but please also add release note to > > `clang-tools-extra/ReleaseNotes.rst` so that people know about the fix:) > > Forgot to mention it before.. sorry. All user-visible changes should go > > with release notes I'm not sure if we need a release note on clang side > > (not familiar how things are done) > > Oh, I forgot to add a release note. Thanks for your feedback! > > @ChuanqiXu9 Could you advise me on how to write the ReleaseNote on the Clang > side?
I think you just put at the back as everyone else, but also with github issue number IIRC. For clang-tidy we don't use issue numbers. https://github.com/llvm/llvm-project/pull/201481 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
