seabass011 commented on issue #51521: URL: https://github.com/apache/airflow/issues/51521#issuecomment-2957294030
Hi @lanxinxs @jroachgolf84 @sjyangkevin — I’d like to **take ownership of this fix**. Plan: * ✅ First, reproduce the error on 3.0.1 to confirm the validator path. * 🔧 Patch: if `key is None`, translate to the literal `"return_value"` (restoring Airflow 2 behaviour). * 🧪 Add a unit test covering the legacy case. * 📝 Tweak the docstring so future readers know `None → "return_value"`. Two-click reply: 1. 👍 = “sounds good, go for it” 2. ➡️ Anything else you’d like double-checked (e.g., back-port label for 3.0.3)? Happy to adjust if you see a better approach—just let me know. Thanks for all your work on Airflow! - Sebastian (UTC-7) -- 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]
