yegor256 / cam

Classes and Metriсs (CaM): a dataset of Java classes from public open-source GitHub repositories
http://cam.yegor256.com
MIT License
23 stars 32 forks source link

Add macos runner #342

Closed timur-harin closed 3 months ago

timur-harin commented 3 months ago

@yegor256 In this PR, I added:

Let's see if all will be fine, but because it will run container...

timur-harin commented 3 months ago

@yegor256 I see from documentation that macOS runners do not support container job

Could I create a separate job for macOS to test sudo make install?

Screenshot 2024-05-24 at 15 04 32
timur-harin commented 3 months ago

@yegor256 I created another job for macos only, to check it we need to merge PR #337 and PR #339

yegor256 commented 3 months ago

@timur-harin maybe it's better to use matrix build? https://docs.github.com/en/actions/using-jobs/using-a-matrix-for-your-jobs

timur-harin commented 3 months ago

@yegor256 As you see in this commit, I already tried, problem is in container job

timur-harin commented 3 months ago

@yegor256 I see from documentation that macOS runners do not support container job

Could I create a separate job for macOS to test sudo make install?

Screenshot 2024-05-24 at 15 04 32

@yegor256 Please see again

timur-harin commented 3 months ago

@yegor256 Oh, yes! Now all is done!

In this separate job for macOS with make install:


- in macOS JPEEK installed in `/opt/app/jpeek.jar`, so we need `sudo` to `mkdir` and `mv`

But this job takes a lot, will you manage to run this job only when you need?

To make this job works with current source code for `make test`  - it will take a lot  
timur-harin commented 3 months ago

@yegor256 please check

yegor256 commented 3 months ago

@timur-harin good work, thanks!