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

chore: update deprecated goreleaser option. #1559

Merged
merged 2 commits into from
Jul 19, 2023

Conversation

winder
Copy link
Contributor

@winder winder commented Jul 7, 2023

Summary

Replaces the deprecated replacements option according to recommendation in docs: https://goreleaser.com/deprecations/#archivesreplacements

I don't recall why the old template had a replacement for Arm. @Eric-Warehime do you know if it's needed?

@winder winder requested a review from Eric-Warehime July 7, 2023 15:06
@winder winder self-assigned this Jul 7, 2023
@winder winder added the Bug-Fix label Jul 7, 2023
@codecov
Copy link

codecov bot commented Jul 7, 2023

Codecov Report

Merging #1559 (d6fdd11) into develop (61f0d13) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1559   +/-   ##
========================================
  Coverage    68.58%   68.58%           
========================================
  Files           37       37           
  Lines         7417     7417           
========================================
  Hits          5087     5087           
  Misses        1903     1903           
  Partials       427      427           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Eric-Warehime
Copy link
Contributor

@Eric-Warehime do you know if it's needed?

Not at all.

.goreleaser.yaml Outdated
Comment on lines 98 to 100
{{- else if eq .Arch "windows" }}Windows
{{- else if eq .Arch "linux" }}Linux
{{- else if eq .Arch "darwin" }}Darwin
Copy link
Contributor

Choose a reason for hiding this comment

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

It looks like based on the docs you linked you don't need overrides for these?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You're right, these are not needed.

@winder winder merged commit 4526a62 into algorand:develop Jul 19, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants