faif/python-patterns

Add Protocol to factory pattern

mfurquimdev opened this issue · 1 comments

mfurquimdev commented

Hi 👋🏽

I see we have some type hints that's been continuously added as mentioned at #373.

As the factory pattern has the constrain of both classes having to implement a localize method, I thought it might be a good idea to reinforce this constrain by using Protocol.

I'd gladly implement this if it's okay.

Here's the PEP-0544 and the Python docs for Protocols.

faif commented

Hi,

Sure, go ahead.

  • 👍1