PaulRBerg / foundry-multibuild

GitHub Action for building a Foundry project with a range of Solidity versions
https://github.com/marketplace/actions/foundry-multibuild
MIT License
39 stars 3 forks source link

Support Forge flag "--skip script" #3

Open PaulRBerg opened 9 months ago

PaulRBerg commented 9 months ago

Requested by Matt Solomon here.

There are two ways in which this could be implemented:

  1. Add a new skip-script input.
  2. Add a new input called skip that is a string and allows users to pass whatever they want.
    • The advantage of this would be that users could skip other directories, not just test or script
    • The disadvantage is that it would be quite complicated to implement - I need to check if Forge takes a glob path or it has to be a single directory
mds1 commented 9 months ago

forge's --skip does accept globs, and --skip test and --skip script are just aliases for *.t.sol and *.s.sol respectively (i.e. they don't actually skip the test and script folders, just files matching those extensions).

PaulRBerg commented 9 months ago

Fantastic, thanks for the color, @mds1.