abhinav-phi commented on PR #44212:
URL: https://github.com/apache/superset/pull/44212#issuecomment-5688567624
Rebased onto current `master` (`4c27cbd`) — thanks for merging #44288! 🎉
Since the merge revision is now upstream, the interim no-op merge-revision
commit on this branch was dropped automatically during the rebase ("skipped
previously applied commit"), so this PR is back to a single frontend-only
commit: the `smart_date_verbose` year fix with tests, nothing else.
`git merge-tree` of the branch against `master` resolves to exactly one
Alembic head (`e2f3a1b9c640`), and the earlier run already showed the whole
frontend side green on this exact change (all 8 sharded jest suites,
lint-frontend, tsc, pre-commit, test-storybook, bundle-size), so the cancelled
runs should replay green here.
One data point for whoever looks next: in the cancelled attempt the
`docker-build (superset)` job had failed at ~18:44Z *after* the merge-head fix
was already in the merge commit, so that one may be unrelated to the Alembic
issue (build-time failure) — worth a look if it reappears. Everything that
failed in the last wave shows `cancelled` rather than a real failure.
Could someone approve and run the new workflow set? The PR should then be
fully green and ready for review.
--
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]
