beme/abide

add option write to `testdata` instead of `__snapshots__`?

b5 opened this issue ยท 1 comments

b5 commented

Fantastic tool! Love bringing react concepts into go land ๐Ÿ˜„.

While go tool has special treatment for directories & filenames that start with .,_, or testdata, the testdata directory it's considered by some to be idiomatic. I know it's a totally irate thing, but would it be great if I could chose to write snapshots to testdata/__snapshots_. One way might be to export the default hardcoded snapshots var. I'd be happy to submit a PR if you're ok with the idea!

@b5 thanks! And I'm totally for it. Send a PR!

And just thinking through release.... better to support existing workflow or make this the new standard? Seems if this is the idiomatic way, it should also be standard.

In any case, let's go for it & we can eventually transition to it being the standard.