8000 Use automatic version numbers in release builds by ofalvai · Pull Request #216 · bitrise-io/envman · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Use automatic version numbers in release builds #216

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

Merged
merged 1 commit into from
Aug 2, 2023

Conversation

ofalvai
Copy link
Contributor
@ofalvai ofalvai commented Aug 2, 2023

For some reason, this repo was still using hardcoded version strings (while all other binary tool repos use Goreleaser and -ldflags.

This PR aligns the release config to what we have in other repos.

- -X github.com/bitrise-io/envman/version.Commit={{ .FullCommit }}
- -X github.com/bitrise-io/envman/version.BuildNumber={{ index .Env "BITRISE_BUILD_NUMBER" }}

archives:
# GitHub release should contain the raw binaries (no zip or tar.gz)
- format: binary
# Name format should match the curl install script
name_template: "{{ .ProjectName }}-{{ .Os }}-{{ .Arch }}"
replacements:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Deprecated and removed, no longer works.

Copy link
Contributor

Choose a reason for hiding this comment

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

Did change from uppercase to lowercase in case of Darwin/Linux?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, the replacements option got deprecated and the suggested new way of doing things is with the name_template + Go templating

Copy link
Contributor

Choose a reason for hiding this comment

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

Right, missed the "title" part from the template.

@ofalvai ofalvai merged commit 1300c57 into master Aug 2, 2023
@ofalvai ofalvai deleted the ofalvai/fix-version-number branch August 2, 2023 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0