-
Notifications
You must be signed in to change notification settings - Fork 15.9k
AIP-84: alias dag_display_name for BackfillResponse
#50014
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Fixing test currently. |
|
I just converted to draft, so we don't get notified on each commits / updates until this is ready. Feel free to convert back to |
3ae76bf to
dfae172
Compare
|
Sorry for bothering. I would turn it back to |
dfae172 to
0dd2bac
Compare
BackfillResponsedag_display_name for BackfillResponse
0dd2bac to
73e34ed
Compare
|
It seems like the CI error is not relevant to my changes. |
pierrejeambrun
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, you'll need to rebase for the CI to be green. (recently we can't force merge red CI even if failure are unrelated)
73e34ed to
ee64f17
Compare
|
Sure, thanks for your review. I have rebased~ |
ee64f17 to
c39f756
Compare
|
Thanks for the review~ |
Related Issue
#46730
How
dag_display_nameforBackfillResponse^ 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.