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

Update case of binary in goreleaser #1

Merged
merged 1 commit into from
Jul 15, 2022

Conversation

accorvin
Copy link
Contributor

We hit an issue when attempting to use this plugin as installed from the
GitHub release archive. It appears that Kustomize expects the case of
the plugin's binary to match the Kind in the hashannotator object.
This change updates the goreleaser.yaml file to create a binary with the
correct tense.

@accorvin
Copy link
Contributor Author

@pcjun97 very cool plugin! What do you think of this change?

@pcjun97
Copy link
Owner

pcjun97 commented Jul 14, 2022

Hey @accorvin, thanks for noticing this issue and opening a PR for it! I prefer if the project_name field is updated to use the correct casing instead, wdyt?

project_name: hashannotator

Something like:

project_name: HashAnnotator
builds:
  - binary: '{{ .ProjectName }}'
archives:
  - id: latest
    name_template: '{{ tolower .ProjectName }}_latest_{{ .Os }}_{{ .Arch }}'

@accorvin
Copy link
Contributor Author

Thanks @pcjun97! I had the same thought. I've never used goreleaser before so I wasn't sure if changing the project name would cause any other unexpected issues so wanted to get your input first.

Updated.

We hit an issue when attempting to use this plugin as installed from the
GitHub release archive. It appears that Kustomize expects the case of
the plugin's binary to match the `Kind` in the hashannotator object.
This change updates the goreleaser.yaml file to create a binary with the
correct tense.
@pcjun97
Copy link
Owner

pcjun97 commented Jul 15, 2022

LGTM!

@accorvin
Copy link
Contributor Author

@pcjun97 done.

@pcjun97 pcjun97 merged commit 8d313aa into pcjun97:master Jul 15, 2022
@accorvin
Copy link
Contributor Author

Thanks @pcjun97! Do you know if a new release will automatically be produced?

@pcjun97
Copy link
Owner

pcjun97 commented Jul 17, 2022

@accorvin Sorry for the delay. You can find the latest release here: https://github.com/pcjun97/kustomize-hash-annotator/releases/tag/1.0.1

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