-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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: bake inso-cli on older macos [INS-3931] #7471
Conversation
6fbcc71
to
876dbd5
Compare
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.
will this make an additional artifact?
if so with macos-13 build run on latest?
if thats true, then should we use oldest stable ubuntu build as well to satisfy that crowd?
|
Is there a reason it needs to also run the macos-14 build? Its not good to support multiple builds when one will suffice. Makes things less intuitive. |
@jackkav we can make it macos-13 only as its better for compatibility |
4fa9ece
to
dcd6f2b
Compare
This reverts commit dcd6f2b.
* chore: bake inso-cli on older macos [INS-3931] * send PKG_NAME var for macos builds * macos-13 only * Revert "send PKG_NAME var for macos builds" This reverts commit dcd6f2b. * Update .github/workflows/release-start.yml
Context:
macos-latest
now defaults tomacos-14
that uses apple sillicon. When we bake inso cli on apple sillicon we screw compatibility with all other macos versions that run on intel.In a follow-up PR we can do something similar for ubuntu
Closes INS-3931
Tested in https://github.com/Kong/insomnia/releases/tag/v9.3.0-alpha.0-experimental
Related pr - #7469 where we tested this