Loosen type requirement for insert_many()?

[mikeckennedy]

Hey. This is low priority, however, could be worth doing.

On Document.insert_many the parameter documents is of type list[Document]. Would it be reasonable to make that any iterable of document rather than strictly list? For example, would a tuple or set of documents work?

There are probably other, related methods with the same situation taking a list of documents. For example, on beanie_init the models list there is also a list rather than iterable.

[roman-right]

Hi @mikeckennedy ,
You are right. I have to make an inspection and replace lists with iterable type

[mikeckennedy]

Thanks Roman. I appreciate that. It's no big deal but something that come up as I'm working on the new course. :)

[roman-right]

Types mistakes in libraries can annoy more than runtime bugs, as IDEs highlight them prominently :D Thank you for the catch!

[roman-right]

It will be fixed in #669

[github-actions]

This issue is stale because it has been open 30 days with no activity.

[github-actions]

This issue was closed because it has been stalled for 14 days with no activity.