pengwk commented on PR #21769: URL: https://github.com/apache/superset/pull/21769#issuecomment-4770759545
Thanks again for rebasing this. I pushed a small follow-up commit to align the saved query integration tests with the intended all-query-access behavior and to fix the export command tests that were failing because the permission check reads from \. Local verification on the updated branch: \\............ [100%] 12 passed in 1.98s\ I also ran changed-file pre-commit successfully for the modified Python files. Full \ currently hits an unrelated existing mypy error in \, outside this PR's files. I marked the PR ready for review. The full GitHub Actions workflows are currently showing \ with no jobs, so they likely need maintainer approval to run on this fork PR. I also do not have permission to remove the stale \ label. Could a maintainer approve the workflows and remove that label? -- 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]
