Skip to content

Refactor: Simplify some loops#34223

Merged
potiuk merged 1 commit intoapache:mainfrom
eumiro:loops-20230908
Sep 15, 2023
Merged

Refactor: Simplify some loops#34223
potiuk merged 1 commit intoapache:mainfrom
eumiro:loops-20230908

Conversation

@eumiro
Copy link
Contributor

@eumiro eumiro commented Sep 8, 2023

No description provided.

@boring-cyborg boring-cyborg bot added area:API Airflow's REST/HTTP API area:dev-tools area:logging area:providers area:Scheduler including HA (high availability) scheduler provider:elasticsearch labels Sep 8, 2023
@eumiro eumiro marked this pull request as ready for review September 11, 2023 18:02
@eumiro eumiro force-pushed the loops-20230908 branch 2 times, most recently from fd3cde5 to 6047b2c Compare September 11, 2023 19:14
Copy link
Member

Choose a reason for hiding this comment

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

I think the count_prs variable is always the same as limit_pr_count (except for 0 but that value doesn’t make practical sense anyway)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

limit_pr_count can be None and we want to print the count_prs anyway, so I changed it back to real len(prs) if limit_pr_count is None.

@potiuk potiuk merged commit d29c348 into apache:main Sep 15, 2023
@eumiro eumiro deleted the loops-20230908 branch September 15, 2023 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants