argoproj / applicationset

The ApplicationSet controller manages multiple Argo CD Applications as a single ApplicationSet unit, supporting deployments to large numbers of clusters, deployments of large monorepos, and enabling secure Application self-service.
https://argocd-applicationset.readthedocs.io/
Apache License 2.0
586 stars 279 forks source link

Include SHA of test merge commit in PR generator for GitHub #623

Open purajit opened 1 year ago

purajit commented 1 year ago

Component: Pull Request Generator

Request: Currently, when using the PR Generator with GitHub, we are provided with the head_sha. Additionally being provided with merge_commit_sha would be extremely useful.

Motivation: The default behavior of a lot of widely-used GHAs, such as actions/checkout and docker/metadata-action, is to use the test merge commit sha rather than the head sha. Providing this value would mean users have a way of working with PR generator without having to override this behavior throughout their stack, and seeing predicatable values being propagated everywhere.

I'm unfamiliar with GitLab and Gitea but I think based on some quick searching that they too create test merge commits, but at least for GH, it seems like it would require a change

Other links: MergeCommitSHA in the GitHub client Argo docs for template and params Information about test merge commit SHA and how it changes