Conversation
This reverts commit f28047a.
|
Please undo these change @auvipy . |
|
Please do not remove infra stuff without making sure it's not used and as it was me who added it so I was expecting a review request before merge. You have now broken my regular security checks. P.S |
|
They reached me first and I refused their request. But later I saw you added this without letting me know or even discussing with me first. Always inform me first to take any important decision regarding the project. you must have raise a discussion first in the forum before adding anything like this. |
This is not correct @auvipy. I did inform in the core-developer discussion before it was archived, you just didn't see it: https://github.com/orgs/celery/teams/core-developers/members/archived_team_posts.json Second, I took @thedrow's word as enough, as I should, because I also notified the core team and there were not objections from anyone. |
|
In my previous message there I tagged you personally @auvipy and you also didn't respond: |
|
The bottom line is that we need to act as a team and not override/remove each other's effort without confirming (like we did correctly with the changerelease workflow removal recently). I try to keep you @auvipy and @thedrow informed and also try to make sure I see your own pings/notifications for me. Some may be lost, like the Semgrep notice you didn't see but I try to avoid repeating my mistakes and you already asked me once to notify you for big changes so I do. We are not here to compete, we are here to contribute. |
|
I might not be clear with my message, I meant to say it was actually not working, so didn't feel important. also the CI was showing that it is actually not doing anything. It is not to undo any team members effort or anything like that. I didn't see that because most probably that was archived automatically before I notice. |
Reverts #8201
This was not doing anything. And we already have other tools enabled