Skip to content

fix invalid version in homebrew-tap #26

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
Jul 1, 2024
Merged

fix invalid version in homebrew-tap #26

merged 1 commit into from
Jul 1, 2024

Conversation

alexrashed
Copy link
Member

Motivation

In localstack/homebrew-tap#3 @mcous raised the issue of an invalid version detection for the LocalStack CLI homebrew tap. They also pointed towards a potential source of the issue: Justintime50/homebrew-releaser#39
Thanks a lot for the pointer! This PR addresses this issue by directly setting the version in the homebrew-releaser.

Changes

@alexrashed alexrashed added the bug Something isn't working label Jul 1, 2024
@alexrashed alexrashed requested a review from silv-io July 1, 2024 13:09
@alexrashed alexrashed self-assigned this Jul 1, 2024
alexrashed added a commit to localstack/homebrew-tap that referenced this pull request Jul 1, 2024
Explicitly sets the version in the homebrew tap to avoid issues
with homebrew wrongly detection the version "64".
This will be fixed for upcoming releases with
localstack/localstack-cli#26.
This commit updates the version for the current release.

Fixes #3.
Copy link
Member

@silv-io silv-io left a comment

Choose a reason for hiding this comment

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

LGTM!

@alexrashed alexrashed merged commit 0fee989 into main Jul 1, 2024
4 of 5 checks passed
@alexrashed alexrashed deleted the fix-version branch July 1, 2024 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants