mikebridge commented on PR #44370: URL: https://github.com/apache/superset/pull/44370#issuecomment-5725053950
Pushed the reviewed master resync and H1/H2 disposition in da066aec93c09154c2b3a90e4a82aaf1b3ea5309. - H1: hoisted the reopened fixture; full `npm run type` and source-resolved TypeScript both pass. - H2: documented dynamic group-by as unsupported on versioned views, corrected the error guidance, and tested the card's actual unstamped mask. Marker-preserving target rebuilds and source-stamped masks remain a follow-up. - Verification: 565 backend tests; 13 Jest suites, 176 passed/1 existing skipped; all applicable scoped hooks, including MyPy and frontend types, pass. The three low review findings are listed in the PR body. This remains draft for the browser walkthrough. Apache master advanced after the reviewed base to 394bca55c792b7b3547e23f6e175a7cb0f0757e8; a read-only merge check finds a new conflict in `superset-frontend/src/dataMask/reducer.test.ts`. Those two later MCP commits were not folded into this verified snapshot, so another scoped resync is still needed. -- 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]
