turbaszek commented on a change in pull request #16268:
URL: https://github.com/apache/airflow/pull/16268#discussion_r645740670
##########
File path: tests/jobs/test_scheduler_job.py
##########
@@ -3921,7 +3921,7 @@ def
test_do_schedule_max_active_runs_upstream_failed(self):
schedule_interval='@once',
max_active_runs=1,
) as dag:
- # Can't use DummyOperator as that goes straight to success
+ # Can't use NoOpOperatoras that goes straight to success
Review comment:
```suggestion
# Can't use NoOpOperator as that goes straight to success
```
And in other places - it seems to be badly performed replacement. In general
I would suggest to separate this PR into two PRs:
- replace non-inclusive words
- rename `DummyOperator` - this change is a big one and we should probably
first reach consensu on the new name
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
