chore: run pre-commit on all files#10119
Conversation
|
GNU testsuite comparison: |
|
Most of the examples in the PR are there on purpose such as the whitespace files for integration tests |
5f06b75 to
311a830
Compare
|
GNU testsuite comparison: |
|
How does it work with the patches files, since they have to match what existed before don't they have to keep all formatting issues? Might be a good idea to exclude all of those ones too |
The formatting removed trailing whitespace in existing blank lines in patch files, so it shouldn't affect testing (CI also passes). The localization CI error seems unrelated. |
311a830 to
4f7df84
Compare
|
@ChrisDryden, could you PTAL again? Only whitespace is changed in the patch files. The |
b745005 to
41817f8
Compare
|
GNU testsuite comparison: |
41817f8 to
07a5351
Compare
|
GNU testsuite comparison: |
07a5351 to
8e90863
Compare
|
GNU testsuite comparison: |
8e90863 to
9ad3108
Compare
|
GNU testsuite comparison: |
|
hi @sylvestre is this good to merge? CI error seems unrelated.. |
9ad3108 to
93b6b04
Compare
|
GNU testsuite comparison: |
Excluded test fixtures.