gkneighb commented on PR #40399: URL: https://github.com/apache/superset/pull/40399#issuecomment-4789223429
Update (`5e0e2e3294`): the approved run surfaced two more master-merge drift issues beyond the first round — now fixed: - **Python-Unit** → master's `test_privacy.py` requires user-directory fields be absent from the published schema; the merge had kept `created_by`/`changed_by` on `DashboardInfo`. Removed them (schema + serializer) to match master's other Info schemas. - **pre-commit / ruff-format** → reformatted with the pinned **ruff 0.9.7** (the prior commit used a newer local ruff that 0.9.7 disagreed with). Full `mcp_service` suite green locally (**2279 passed**). @aminghadersohi — sorry for the extra round; could you **re-approve the workflow run** once more? Should be clean now. 🙏 -- 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]
