gaurav0107 commented on PR #42489: URL: https://github.com/apache/superset/pull/42489#issuecomment-5097510245
Heads up on CI, since this branch has stale queued runs on three earlier SHAs. The workflow `concurrency` group is keyed on PR number rather than head SHA with `cancel-in-progress: true`, so approving runs for more than one SHA makes them cancel each other. That is what left `unit-tests-required`, `test-postgres-required`, `cypress-matrix-required` and `playwright-tests-required` red a moment ago — the same trap the header comment in `pr-lint.yml` describes. Approving only the runs on head `39a84f5` should avoid it. For reference, Frontend Build CI did complete green on `8275e97`: all 8 `sharded-jest-tests` shards, plus `lint-frontend`, `validate-frontend`, `test-storybook` and `report-coverage`. The only change since then is reordering one conjunct inside `checkForHtml` so a non-string `message` keeps its existing `isJsonString` short-circuit. -- 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]
