[libcxx] [llvm] [docs] Fix malformed csv files (PR #80567)

via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 6 16:36:21 PST 2024


wusatosi wrote:

> IMO this makes sense, but could we add the CSV validation in this PR? Otherwise there’s no « testing » for this change.


Technically you can check this pr by just looking at the csv file on GitHub

Broken (current):
https://github.com/llvm/llvm-project/blob/1b03cbc93989c84ad0b78c27d4427a7eaa5842f1/libcxx/docs/Status/FormatPaper.csv
Fixed:
https://github.com/wusatosi/llvm-project/blob/91963d8d02ee1cffea6b0a1da2f9b5bee2b08c4f/libcxx/docs/Status/FormatPaper.csv

But yes this definitely needs to be in the CI.

What is the standard for adding code-format check in CI? Should I add this functionality as a stand-alone CI file, or should I add this into `code-format-helper.py`?

https://github.com/llvm/llvm-project/pull/80567


More information about the llvm-commits mailing list