-
Notifications
You must be signed in to change notification settings - Fork 58
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
df1491c
commit ac8ac4a
Showing
1 changed file
with
1 addition
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ac8ac4a
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👋 is this your latest release? I notice that it's not showing as the latest release here on GitHub yet (which shows 1.2.1).
Just checking for Homebrew/homebrew-core#74114.
ac8ac4a
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@carlocab thanks for checking. So it looks like brew picks up the the latest tags before I do a GH release. This is what I see in the releases page:
I plan to update the releases on GH, but the timing might be a bit off (when I updated these tags it was for a different package manager release).
ac8ac4a
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops. It looks like we're on 1.3.0 now. Should we switch back to 1.2.1?
ac8ac4a
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No I think the right thing to do here is for me to just publish a release on GH for those tags, including 1.4.0 (and the PR for Homebrew/homebrew-core#74114 is good 👍 )
ac8ac4a
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @rvantonder. That's actually a really big help. Downgrading formulae for us is rather tricky.
ac8ac4a
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good! I'm going to try automate things so that the GH releases are synced with tags for future. Thanks again.