andreydevyatkin commented on PR #29026: URL: https://github.com/apache/beam/pull/29026#issuecomment-1773798448
> Sorry for the slow review, this looks good to me, I think standardizing on setup is a good idea. Once conflicts are resolved it should be good to merge @damccorm Actually these changes are more than just standardization (lack of description for PR), I would say that it is a bonus to the actual intent. The purpose of adding the `setup-environment-action` action is to publish gradle snapshots - there is a baked-in gradle setup step that must be added to every workflow file. Initially, I added the `setup-environment-action` step to the `setup-action` action in order to delay applying this change everywhere until we migrated all workflows. Now, it's time to move out this step from `setup-action` because they are about different meaning. -- 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]
