-
Notifications
You must be signed in to change notification settings - Fork 3
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
Removing "awaiting response" label from adapters #35
Comments
First, 100% agree this needs to be added to adapters. That's not the purpose of my comment. My comment is more generally process related. Tagging @leahwicz and @lostmygithubaccount to see if they have opinions here. This is a good time to have the discussion on when to centralize actions. This action is just a single step job to a marketplace action
It may be worth centralizing so that if we change the labels for some reason we do it in only one place. Or if we update the version of the actions we want to use (we really shouldn't blindly point to But do we want to centralize workflows that are single step workflows like this? There are others that we could centralize that I have not because I was trying to avoid layering too deep but I'm not sure that's a great reason. When you make it a reusable workflow the workflow steps show in your local run so all the debug info is available, unlike when you're using a marketplace action and it's a bit of a blackbox. After typing this up, I'm leaning towards centralizing anything we use across core and adapters if for no other reason than to control the versions we're using in a singular place. |
Noting here so it's not lost: If we standardize on centralized actions, we should also pull the changie-bot action here and refer to it in all the adapters and core. There's also multiple issues open to update that action in all the adapters (minus spark which was already done). |
Thanks for your analysis @emmyoop ! Definitely keen to support whatever your recommendation is. |
I'm not a huge fan of using random people's actions. all this is really doing is making a GitHub API request to change a label -- I think that can be done with the I suspect that could help us more easily customize the behavior as we see fit too even if we don't blindly point to |
Couple points here:
|
I agree with @lostmygithubaccount - using the Some marketplace actions are super powerful though. Evaluating on a case by case basis is worthwhile. That should include actually looking at the code too. I have read that some places approach this by forking those actions and pointing to their own fork to fully control its version. There is a security concern with pointing at I suspect getting this specific action into the adapters might not wait until the contractor comes on since it effects monitoring triage. I definitely think it needs to be centralized here as part of the work to send it everywhere. It's not much more work than adding each one individually. Once it's centralized we can rework what it's doing specifically then if we want to wait for the contractor to help us make the best call there. |
Quickest solution
Copy this action over to each adapter plugin repos
Best solution
The text was updated successfully, but these errors were encountered: