warren830 commented on PR #4607:
URL: 
https://github.com/apache/incubator-devlake/pull/4607#issuecomment-1465429679

   > > > Great Job.
   > > > Two more comments:
   > > > 
   > > > 1. Suggest adding this stage as the last stage in build.yml, as we 
need to ensure the images are pushed success, then trigger the helm chart 
promote version.
   > > > 2. Previously, only actions from verified creators are allowed in this 
repo.
   > > 
   > > 
   > > Thank you for you careful review! @matrixji
   > > For the second suggestion, I want to use the way above to sovle it:
   > > Using`git submodule` to download actions to `.workflows/actions` in 
`apache/incubator-devalke`, like the solutions in helm chart:
   > > 
https://github.com/apache/incubator-devlake-helm-chart/tree/main/.github/actions
   > > <img alt="image" width="370" 
src="https://user-images.githubusercontent.com/36830265/223626378-69caea4e-7e07-4b33-a04d-4e18966141ad.png";>
   > > Is it allowed?
   > 
   > IMO, this it's acceptable, @warren830 could also help comment on that?
   
   
   I'm not quite sure about the workflow of the second suggestion. Will this 
also submit a pr or promote automatically?
   From my perspective, creating a github bot to submit pr, and manually 
approving it by us is acceptable, but if the second proposal require less work, 
I think it'd be better.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to