potiuk commented on PR #32218: URL: https://github.com/apache/airflow/pull/32218#issuecomment-1610927650
> It is unclear to me why a metaclass is needed. From what I can tell this can be most simply solved by calling `__init__` in `ProvidersManager.__new__` (and `ResourceVersion` doesn’t need initialisation to begin with) I think there are various approaches, I actually like the metaclass approach, it's cleaner as it also allows to avoid duplication. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
