1
0
mirror of https://github.com/goreleaser/goreleaser.git synced 2025-03-17 20:47:50 +02:00

feat(krew): sync fork before opening PR

Signed-off-by: Carlos Alexandro Becker <caarlos0@users.noreply.github.com>
This commit is contained in:
Carlos Alexandro Becker 2024-03-26 23:04:38 -03:00
parent b9b8a65618
commit 2ecd71ee71
No known key found for this signature in database
2 changed files with 27 additions and 11 deletions

View File

@ -335,25 +335,36 @@ func doPublish(ctx *context.Context, manifest *artifact.Artifact, cl client.Clie
return err
}
if !cfg.Repository.PullRequest.Enabled {
return cl.CreateFile(ctx, author, repo, content, gpath, msg)
base := client.Repo{
Name: cfg.Repository.PullRequest.Base.Name,
Owner: cfg.Repository.PullRequest.Base.Owner,
Branch: cfg.Repository.PullRequest.Base.Branch,
}
log.Info("brews.pull_request enabled, creating a PR")
pcl, ok := cl.(client.PullRequestOpener)
if !ok {
return fmt.Errorf("client does not support pull requests")
// try to sync branch
fscli, ok := cl.(client.ForkSyncer)
if ok && cfg.Repository.PullRequest.Enabled {
if err := fscli.SyncFork(ctx, repo, base); err != nil {
log.WithError(err).Warn("could not sync fork")
}
}
if err := cl.CreateFile(ctx, author, repo, content, gpath, msg); err != nil {
return err
}
return pcl.OpenPullRequest(ctx, client.Repo{
Name: cfg.Repository.PullRequest.Base.Name,
Owner: cfg.Repository.PullRequest.Base.Owner,
Branch: cfg.Repository.PullRequest.Base.Branch,
}, repo, msg, cfg.Repository.PullRequest.Draft)
if !cfg.Repository.PullRequest.Enabled {
log.Debug("krews.pull_request disabled")
return nil
}
log.Info("krews.pull_request enabled, creating a PR")
pcl, ok := cl.(client.PullRequestOpener)
if !ok {
return fmt.Errorf("client does not support pull requests")
}
return pcl.OpenPullRequest(ctx, base, repo, msg, cfg.Repository.PullRequest.Draft)
}
func buildManifestPath(folder, filename string) string {

View File

@ -368,6 +368,10 @@ func TestRunPipePullRequest(t *testing.T) {
Branch: "update-{{.Version}}",
PullRequest: config.PullRequest{
Enabled: true,
Base: config.PullRequestBase{
Owner: "og",
Name: "bar",
},
},
},
},
@ -400,6 +404,7 @@ func TestRunPipePullRequest(t *testing.T) {
require.NoError(t, publishAll(ctx, client))
require.True(t, client.CreatedFile)
require.True(t, client.OpenedPullRequest)
require.True(t, client.SyncedFork)
golden.RequireEqualYaml(t, []byte(client.Content))
}