spago icon indicating copy to clipboard operation
spago copied to clipboard

Added main opts for test command

Open imsaravana369 opened this issue 1 year ago • 1 comments

Added main opts for test command, doing spago test --main modulename was failing because somehow the main option got removed in the new spago .

TODO

Checklist:

  • [ ] Added the change to the "Unreleased" section of the changelog
  • [ ] Added some example of the new feature to the README
  • [ ] Added a test for the contribution (if applicable)

imsaravana369 avatar Feb 27 '24 17:02 imsaravana369

and is creating a separate pr a overkill here? Not sure about the convention

imsaravana369 avatar Feb 27 '24 17:02 imsaravana369

Separate PR is good practice here, since every one of them is squashed in a single commit and it's good to review unrelated patches separately.

f-f avatar Mar 01 '24 07:03 f-f