vincbeck commented on code in PR #55736:
URL: https://github.com/apache/airflow/pull/55736#discussion_r2353443999
##########
airflow-core/src/airflow/dag_processing/bundles/manager.py:
##########
@@ -152,8 +166,6 @@ def parse_config(self) -> None:
If a bundle class for a given name has already been imported, it will
not be imported again.
- todo (AIP-66): proper validation of the bundle configuration so we
have better error messages
Review Comment:
Hey @jedcunningham. When working on this one I took the liberty to use
Pydantic to specify the dag bundle config schema. Let me know if this is what
you meant by "proper validation of the bundle configuration so we have better
error messages". Happy to revisit if you had something else in mind.
--
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]
