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

Don't put __snapshots__ in project root #97

Open
aaronjameslang opened this issue Jun 1, 2018 · 2 comments
Open

Don't put __snapshots__ in project root #97

aaronjameslang opened this issue Jun 1, 2018 · 2 comments

Comments

@aaronjameslang
Copy link

No accounting for taste, but test data should be in /test or the project's equivalent

Ideally we'd detect the test directory from mocha/etc, but it could also be configurable

Would this change be welcome? Are there technical limitations an implementer should be aware of?

@kuceb kuceb changed the title [enhancement] Don't put __snapshots__ in project root Don't put __snapshots__ in project root Jun 1, 2018
@kevzettler
Copy link

This should definitely be a config value or something

@hexsprite
Copy link

I tend to like the this directory to be stored in the same base directory as the test file in question. This is consistent with how Jest is configured as well I believe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants