shoemoney commented on PR #44460: URL: https://github.com/apache/superset/pull/44460#issuecomment-5771914126
Heads up on the red CI here — the `unit-tests` / `babel-extract` failures are pre-existing drift on `master`, not something this PR introduces. `tests/unit_tests/scripts/translations/check_pot_drift_test.py` fails on 5 strings missing from `superset/translations/messages.pot`: - `Configure retries when alert or report generation fails before delivery.` - `Contact the report owner for error detai…livery failures are not replayed.` - `Screenshot cache is unavailable` - `Screenshot request is temporarily unavailable` - (one more in the same alert/report + screenshot group) None of those strings appear in this diff (this change is confined to `granularity_sqla` resolution in the query context), and `master` at 22d37f3 does not contain them in `messages.pot` either — so any PR branched off current master hits the same failure. Whoever landed the alert-retry / screenshot-cache copy needs to regenerate the `.pot`. Happy to regenerate it here if you'd rather unblock this PR that way, but it would add unrelated translation churn to the diff, so I've left it alone. Let me know which you prefer. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
