njames93 added inline comments.
================ Comment at: clang-tools-extra/docs/ReleaseNotes.rst:221-227 Removed checks ^^^^^^^^^^^^^^ Improvements to include-fixer ----------------------------- The improvements are... ---------------- LegalizeAdulthood wrote: > salman-javed-nz wrote: > > Small nit, not about this patch, but the documentation release process as a > > whole. > > > > Clang 13 got released with a lot of these placeholders still in the > > documentation. > > https://releases.llvm.org/13.0.0/tools/clang/tools/extra/docs/ReleaseNotes.html > > They were never tidied up before release. > > > > What could be done to ensure that this doesn't happen again? Perhaps keep > > the placeholders in the file but in a commented-out state? > > > > I'm aware of > > https://llvm.org/docs/HowToReleaseLLVM.html#update-documentation but in > > this case I guess it wasn't enough to capture this? > I think it would be reasonable to have them as comment blocks. > > I don't know if there's an automated process for generating/updating > release notes. I was always under the impression those got removed when the release branch is created Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118519/new/ https://reviews.llvm.org/D118519 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
