Zulip Chat Archive
Stream: FLT
Topic: CI update
Kevin Buzzard (Apr 26 2025 at 14:36):
I've spent the day working on CI for FLT. We now have activated a bunch of linters in CI which will check for things like: FLT.lean
not being a list of the files in the project, lines being more than 100 characters long, definitions not having docstrings and so on. I did a bunch of work on the project today which caused several PRs to pick up merge conflicts; I fixed the conflicts because I could see how I'd caused them, but in the process of merging master I of course triggered the new CI, so I'm really sorry @Salvatore Mercuri (FLT#414) and @Matthew Jasper (FLT#432), I broke your PRs! Hopefully it's nothing worse than having to add a few docstrings, but another change was that I streamlined imports so there is a chance that you might have had to add some back to get things working again. Looks like all the Lean code is compiling now at least...
Matthew Jasper (Apr 26 2025 at 14:48):
FLT#432 is working again
Kevin Buzzard (Apr 26 2025 at 14:58):
Thanks and sorry! I'll review it :-) I've had enough of infrastructure for one day...
Salvatore Mercuri (Apr 27 2025 at 10:34):
FLT#414 also now passing!
Last updated: May 02 2025 at 03:31 UTC