feat: verify dag details e2e test#59616
Conversation
|
Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contributors' Guide (https://github.com/apache/airflow/blob/main/contributing-docs/README.rst)
|
|
Nice!. I will review soon |
choo121600
left a comment
There was a problem hiding this comment.
just nit suggest :)
We’ve decided to use ‘Dag’ instead of ‘DAG’.
results as of https://lists.apache.org/thread/0651j4vdgzmlhgndmotvznlg97kyh328
|
And if you include the issue in the PR description, it will make it easier for reviewers to review 😉 |
For some reason the PR is not getting linked to the issue even after adding closes and related issue number |
@iharsh02 I have updated the description with closing and related issue its fine now |
Co-authored-by: Yeonguk Choo <[email protected]>
Co-authored-by: Yeonguk Choo <[email protected]>
|
Awesome work, congrats on your first merged pull request! You are invited to check our Issue Tracker for additional contributions. |
|
Cooool🎉🎉🎉 |
feat: verify dag details e2e test
feat: verify dag details e2e test
feat: verify dag details e2e test
E2E test to verify DAG details
Closes: #59555
Related: #59028
^ 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.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.