o-nikolas commented on PR #51326: URL: https://github.com/apache/airflow/pull/51326#issuecomment-2932517119
> CI is failing because caplog is used in some of the tests I moved. I already have a follow-up PR queued with those changes built on top of this one, but I wanted this one to be just a straight copy/paste without logic changes. What do you folks think? Can that wait until this one is merged in order to keep the two PRs more focused, or should I just combine that work in here? We shouldn't knowingly push it if we know it's going to fail CI in main. But maybe put up the second PR that includes these changes _and_ the test fix as separate commits? Then if someone wants to see just the move they can look at that commit in isolation? -- 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]
