Remove old TravisCI config file#978
Conversation
As far as I can tell, TravisCI is no longer used in this repo. Signed-off-by: Asher Foa <1268088+asherf@users.noreply.github.com>
|
Welcome @asherf! |
|
Hi @asherf. Thanks for your PR. I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
Note: FSx for Lustre did this as well in this PR: kubernetes-sigs/aws-fsx-csi-driver#302. @asherf you should also remove the test coverage check in the readme: https://github.com/kubernetes-sigs/aws-efs-csi-driver/blob/master/docs/README.md?plain=1#L1 |
|
@asherf Sorry for taking so long to come back to this, can you rebase those 3 commits into one? Once that's done I can help this get merged. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: asherf The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
done. |
|
Can you squash the 4 commits into one? |
|
afaik the preferred approach for reviews is to have the cli bot perform the merge, part of the reason being that an admin for the repo is needed to merge the pr in this way. I hope you don't mind, I rebased your commit for you and opened a new PR: #1129 |

As far as I can tell, TravisCI is no longer used in this repo.
Feel free to close this PR if travisCI is still being used and this file should remain