-
Notifications
You must be signed in to change notification settings - Fork 5.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Removed un-necessary workflows #28449
fix: Removed un-necessary workflows #28449
Conversation
Hey, @vedpatwardhan. I removed all the workflows that I think are not useful now. I'm not 100% sure about the following 5, but I removed them, so please let me know your opinion on these.
Thank you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey, @vedpatwardhan. I removed all the workflows that I think are not useful now. I'm not 100% sure about the following 5, but I removed them, so please let me know your opinion on these.
- label_discussion_comment.yml
- manual-tests.yml
- manual-tests-pr.yml
- run-failing-tests.yml
- test-ivy-cron-gpu.yml
Thank you.
Along with these 5, there's a few more that I've flagged that should be retained, rest can be removed. Thanks @Sai-Suraj-27 😄
.github/workflows/github_release.yml
Outdated
@@ -1,34 +0,0 @@ | |||
name: github-release | |||
on: | |||
workflow_dispatch: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this workflow should be retained
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done 👍🏻
.github/workflows/pypi.yml
Outdated
@@ -1,32 +0,0 @@ | |||
name: pypi | |||
on: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this workflow should be retained
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done 👍🏻
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm! Feel free to merge the PR, thanks @Sai-Suraj-27 😄
PR Description
Removed un-necessary workflows.
Related Issue
Closes #28447
Checklist
Socials