Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Goreleaser s3 upload not working #1243

Merged
merged 6 commits into from
Jul 23, 2024
Merged

Goreleaser s3 upload not working #1243

merged 6 commits into from
Jul 23, 2024

Conversation

dopey
Copy link
Contributor

@dopey dopey commented Jul 23, 2024

No description provided.

@dopey dopey requested a review from maraino July 23, 2024 04:27
@github-actions github-actions bot added the needs triage Waiting for discussion / prioritization by team label Jul 23, 2024
@dopey dopey requested a review from tashian July 23, 2024 16:10
@dopey dopey merged commit 07a000e into master Jul 23, 2024
16 checks passed
@dopey dopey deleted the max/s3-upload branch July 23, 2024 17:02
ids:
- s3-versioned
- s3-unversioned
bucket: '{{ .Env.AWS_S3_BUCKET }}'
region: us-east-1
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should use an environment variable for this too.

- provider: s3
disable: '{{ if .Prerelease }}true{{ else }}false{{ end }}'
ids:
- s3-unversioned
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, for now, I think we are building the same binaries twice, we could use name_template in the extra_files to change the name of the s3_versioned files.

@hslatman hslatman added this to the v0.27.3 milestone Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs triage Waiting for discussion / prioritization by team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants