-
Notifications
You must be signed in to change notification settings - Fork 67
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
Enabling MacOS launcher artifacts testing in github workflows #491
Enabling MacOS launcher artifacts testing in github workflows #491
Conversation
bb6ca36
to
8d6fa65
Compare
89dceee
to
2fab3ee
Compare
@sravanlakkimsetti kindly review.. MacOS testing was the missing bit in github workflows from launcher perspective and I have added that.. no binary updates required because of this PR.. |
@subyssurendran666 Can you please take a look at this one? |
2fab3ee
to
92535f7
Compare
I noticed on some PRs that the Build Equinox / build-launcher-mac64 is often failing see for example: will this be fixed here as well? |
Sure @sravanlakkimsetti |
92535f7
to
5403b18
Compare
@subyssurendran666 @sravanlakkimsetti can we please finish this? currently the mac build always fails and this seems to fix it. As this only affects the github verification I would give +1 as PL for merging this even in freeze period if not @tjwatson has any concerns. |
the fix for this is to remove this line
For the code changes I approved this change. |
5403b18
to
53da656
Compare
f3d0d62
into
eclipse-equinox:master
Relates to eclipse-equinox/p2#166