go-shiori / shiori

Simple bookmark manager built with Go
MIT License
9.03k stars 546 forks source link

ci: increase bsd test vm cpu count #905

Closed fmartingr closed 2 months ago

fmartingr commented 2 months ago

Increases the CPU count for the OpenBSD test to check if that influences the test total execution time.

fmartingr commented 2 months ago

@pacoesteban Any ideas why the ARM64 VM takes 1hr to run the test suite? I'm reluctant to spent time on this unless it's an easy fix. If that's not the case I would just remove the OpenBSD/ARM64 test (leaving the x86 in there).

pacoesteban commented 2 months ago

I have no idea and, unfortunately, I don't have an ARM64 available to test at the moment. I would say, leave only x86 for now. I'll try to do some tests and get back to you if I find something.

fmartingr commented 2 months ago

Yeah I removed it in #907, there's no rush since that test is optional, but I don't want all CI to last for 1h for each PR 😓 No need to research, just wanted to know if you knew something, thanks for answering.