parkhojeong commented on PR #66211: URL: https://github.com/apache/airflow/pull/66211#issuecomment-4365354289
> LGTM overall! Changes are clean and consistent just need to fix the 8 failing API/CLI tests they are still expecting "DAG" in the assertions. Update those test strings to "Dag" and you are good to go. @Subham-KRLX Thanks for the detailed review ! i found 11 fail cases in github action <img width="1797" height="913" alt="image" src="https://github.com/user-attachments/assets/df24ac7e-4a74-4128-8e12-44932faf0358"; /> i updated test for the fail cases https://github.com/apache/airflow/pull/66211/commits/5742d837af537a6913314d1403f53a27b64bffae here is touched files. <img width="946" height="226" alt="image" src="https://github.com/user-attachments/assets/a0f83d46-97e4-43cf-b1b0-b1e59404dc3d"; /> -- 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]
