fitzee commented on PR #42253: URL: https://github.com/apache/superset/pull/42253#issuecomment-5080988038
Post-fold verification: #42383 is merged/folded into this PR. Its validated commit `54da85d3e9529f3e5facef915d8345581cd9c7b3` was subsequently rebased onto the latest master; the equivalent nine-file patch is now at head `62408beb83fb1a5b56c8f3503803c056a654346c`. All 63 reported checks pass (4 expected skips); review is still required. Scope precision for review: this fixes the #42253 staging regression and hardens non-tiled readiness (positive terminal states, chart-specific outer holder ID/selector excluding Markdown/DynamicComponent, Celery-derived remaining budget with cleanup margin, early/final diagnostics, and fail-before-capture cleanup). It does **not** establish or close the pre-existing PointFive SC-113788 root cause, which predates #42253. Corrected 6.0 staging validation: readiness **5.01s**, full lifecycle **10.52s**, cache **Updated**, and the chart-bearing PDF was visually confirmed. The earlier `SavedQueries.test.tsx` lint failure was an unrelated master regression; the current rebased frontend lint run passes and this PR does not modify that file. -- 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]
