Skip to content

fix release-calendar newlines

Erik Faye-Lund requested to merge kusma/mesa:release-calendar-eol into main

Here's another take on the problem !11854 (closed) and !11779 (merged) tries to fix, but hopefully a bit less heavy-handed.

There's two issues:

  1. The current CSV file contains CRLF instead of LF in the repo. There's many way this could have happened, and this MR doesn't really try to prevent all ways this could happen in the future. See !11779 (merged) for a more complete approach.
  2. The script to update the CSV file opened the output file incorrectly, resulting in bad newlines when run on Windows.

The second problem could very well be the reason why the first problem happened, but we should really look into doing !11779 (merged) as well, as it clearly caught more CRLF files in the tree.

Merge request reports