Zulip Chat Archive
Stream: general
Topic: Further NameSet API
Michael Rothgang (Jan 09 2025 at 12:56):
In #20568, I'm using some further API for NameSet
s (an ofArray
method); ImportGraph's [RequiredModules] file has some more API. I think this would fit well in a more central location.
batteries doesn't have anything about NameSet right now: would a PR adding this to core directly be welcome? CC @Kim Morrison
Kim Morrison (Jan 09 2025 at 13:56):
Please send it to Batteries first. It's had NameSet
in the past, and can again. :-)
Jon Eugster (Jan 09 2025 at 14:08):
if you'd like to create a PR to ImportGraph after you've PRed some API to batteries, that would be very welcomed :)
Michael Rothgang (Jan 09 2025 at 16:46):
Yaël Dillies (Jan 09 2025 at 16:47):
Michael Rothgang (Jan 09 2025 at 17:08):
WIP PR at https://github.com/leanprover-community/import-graph/pull/49: help finishing it, after the batteries PR has been merged, is very welcome :-)
Last updated: May 02 2025 at 03:31 UTC