Is there a reason why all 3 checks are in 1 job?

OpenAstronomy / baldrick

I have a cunning plan!

https://baldrick.readthedocs.io

MIT License

14 stars 8 forks source link

Is there a reason why all 3 checks are in 1 job? #78

Open pllim opened 5 years ago

pllim commented 5 years ago

When one of the jobs fail, it is hard to see immediately which one. Should these be 3 separate jobs in a matrix?

Unit tests
PEP 8
Doc build

astrofrog commented 5 years ago

Trying to reduce ecological footprint from having to spin up 3 VMs for such a small package :) But seriously though, feel free to split.

pllim commented 5 years ago

Now that you guilt trip me like that, I am not sure how to respond. Haha. 🤔

astrofrog commented 5 years ago

Sorry for the 🇬🇧 reply :wink: - actually I don't know if it really matters if we split it or not in terms of build time/energy, I wasn't really being serious above.

pllim commented 5 years ago

Do you wanna try out Azure here? 🤔

pllim commented 5 years ago

I am thinking... if we going to split the job anyway, it is not too much more work to just do it in azure-pipelines.yml. (Not a 🇬🇧 reply.)