Skip to content

Conversation

@ashb
Copy link
Member

@ashb ashb commented Apr 16, 2025

Previously we just ended up with a non-descript "KeyError" message. That is
bad UX.

Screenshot 2025-04-16 at 13 47 18

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

Copy link
Contributor

@amoghrajesh amoghrajesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh boy, thats not nice at all!

Previously we just ended up with a non-descript "KeyError" message. That is
bad UX.
@ashb ashb force-pushed the better-dag-or-task-not-found-errors branch from 16f7f75 to 4a3efc1 Compare April 16, 2025 13:09
@ashb ashb requested a review from mobuchowski as a code owner April 16, 2025 13:09
@kaxil kaxil merged commit 349798c into apache:main Apr 16, 2025
89 of 91 checks passed
@kaxil kaxil deleted the better-dag-or-task-not-found-errors branch April 16, 2025 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants