-
Notifications
You must be signed in to change notification settings - Fork 12
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
try to add arm to github action #106
base: main
Are you sure you want to change the base?
Conversation
Wow, it took 90m to build on macos ??? |
It looked like it built and then stalled waiting for the cleanup to happpen
(between steps).
…On Fri, Jul 26, 2024, 10:12 Jacques Nadeau ***@***.***> wrote:
Wow, it took 90m to build on macos
???
—
Reply to this email directly, view it on GitHub
<#106 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABDDCFDTSQIOE4Z5MPRXCTZOJ7QTAVCNFSM6AAAAABLPRMJZ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJTGE3DGMBXGQ>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
It is odd that the required "build" option is still pending and that build was renamed to linux-build. I don't see an option to change the required builds in the repo though. |
Running again to see if performance continues to be bad.
That's something configured inside the Github UI afaik. I changed the names to be clearer around steps but would need to change at the repo level as well if we decide to merge this. Will wait to see if perf was a random problem or constant. If constant I'm inclined to shelve this for now. |
No description provided.