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]
