potiuk commented on issue #26130: URL: https://github.com/apache/airflow/issues/26130#issuecomment-1235647783
> @kaxil I think attrs not actual superset of dataclasses because dataclasses itself introduced only in 3.7 and dataclasses itself has less abilities than `attrs`. > > My main point more about is it actually required to use `attrs` for extra links or it is possible to use any other types such as `dataclasses` or even regular class. However since we really don't know about third-party extra links we can't replace it. But this does not mean we cannot removed cattrs as required Airlfow dependency. Potentially we could fallback to optionally available cattrs if the custom base link fail to instantiate. We can safely make an assumption that in most cases where somoene uses custom links they already have cattrs installed so this is not going to be a breaking change for them. And even if they (for example) use our image and no cattrs is installed there, we could check it and crash hard with error "please install cattrs to make it works again" -- 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]
