Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

allow to exclude some .purs files from compiling #1203

Closed
srghma opened this issue Mar 17, 2024 · 2 comments
Closed

allow to exclude some .purs files from compiling #1203

srghma opened this issue Mar 17, 2024 · 2 comments

Comments

@srghma
Copy link
Contributor

srghma commented Mar 17, 2024

while I was doing natefaubion/purescript-tidy#122

I found that I need a way to exlude .purs files in lib/test/snapshots folder (those .purs files are not really purescript code)

But cannot accomplish this

2024-03-17-02pm-53-41-screenshot

@srghma
Copy link
Contributor Author

srghma commented Mar 17, 2024

    execArgs:
      - "--exclude-files"
      - "snapshots/*" # TODO: doesnt work
      - "--exclude-files"
      - "test/snapshots/*" # TODO: doesnt work
      - "--exclude-files"
      - "lib/test/snapshots/*" # TODO: doesnt work

doesnt work too

srghma added a commit to srghma/purescript-tidy that referenced this issue Mar 18, 2024
@f-f
Copy link
Member

f-f commented Mar 20, 2024

Spago treats the test folder as the place where to find tests - all the PureScript files in there will be treated as input sources for the test suite of the package.
This means that if you need some fixtures for the tests, as this is the case here, then you shouldn't be including them there. See how in Spago itself we include them in the text-fixtures folder, that is outside of the test tree.

@f-f f-f closed this as not planned Won't fix, can't repro, duplicate, stale Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants