Druid: Run rustfmt CI action only on one host

Created on 8 Oct 2019  路  5Comments  路  Source: linebender/druid

I don't think rustfmt gives different results on different platforms, so runs-on: ${{ matrix.os }} can be shortened to one of the hosts. Saves CI time.

help wanted

All 5 comments

My belief was that rustfmt wouldn't format code that wasn't built on the current platform, but I will investigate!

edit: you're right, it seems to format everything regardless of whether it is built, so we can definitely remove the matrix. 馃憤

Would it be preferable to run this on a Linux platform or on one of two platforms already being used for the rustfmt job?

I _assume_ linux is best for this, there should be more linux resources? Just guessing.

That's what I was also assuming when I went to look at what modifications would be required for this so I was surprised to see that it was running it on macOS and Windows but not any Linux distro which threw me off hence the question.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

MGlolenstine picture MGlolenstine  路  3Comments

insignification picture insignification  路  5Comments

psychon picture psychon  路  6Comments

MGlolenstine picture MGlolenstine  路  6Comments

chris-zen picture chris-zen  路  6Comments