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 NameSets (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):

batterires#1091

Yaël Dillies (Jan 09 2025 at 16:47):

batteries#1091

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