search

mosscomp / moss

A computer for human beings.
https://mosscomp.com
Apache License 2.0
42 stars 2 forks source link

Skip verilator CI step if rtl not modified #20

Closed hasheddan closed 3 years ago

hasheddan commented 3 years ago

We shouldn't run the verilator CI job when changes in a PR do not affect rtl that it tests.

hasheddan commented 3 years ago

Decision has been made to run verilator on every PR for now due to limits of skipping required CI workflows. See https://github.com/mosscomp/moss/pull/22#issuecomment-808914786 for more info.