Closed harshad16 closed 2 years ago
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign sesheta after the PR has been reviewed.
You can assign the PR to them by writing /assign @sesheta
in a comment when ready.
The full list of commands accepted by this bot can be found here.
@harshad16: The following test failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
aicoe-ci/prow/mypy | 40afa2d35558ce48f2d7bbbedc27b85ebf9d9234 | link | true | /test thoth-mypy-py38 |
Full PR test history. Your PR dashboard. Please help us and open an issue when you hit one in your PR.
lgtm, merging as discussed
Related Issues and Dependencies
1250
Description
Testing done we have removed the mypy check for now, will be included in a separate PR.