Zulip Chat Archive

Stream: PR reviews

Topic: Actually removing deprecated modules


Jeremy Tan (Dec 12 2025 at 02:45):

#32765 removes 6-month-old deprecated modules. I remember there's a script to auto-remove deprecated declarations, but it doesn't extend to modules yet

Johan Commelin (Dec 12 2025 at 05:00):

Thanks! I've kicked this on the merge queue.

Johan Commelin (Dec 12 2025 at 05:01):

But I noticed that a bot posted a boatload of warnings about these removed files. We should teach the bot that it shouldn't warn about removing files with a deprecation data > 6 months old.

Ruben Van de Velde (Dec 12 2025 at 06:47):

Dunno if it's worth complicating the bot, it's just a warning that you're entitled to dismiss

Johan Commelin (Dec 12 2025 at 07:13):

With PRs like these it would be helpful. Because it would confirm that indeed all 58 files were deprecated > 6 months ago.


Last updated: Dec 20 2025 at 21:32 UTC