garris / garris/BackstopJS

Avoid writing to node_modules/backstopjs/

Open
#363 5 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
7.2k
Forks
621
PR merge metrics
No merged PRs in 30d

Description

I noticed that backstopjs is creating temporary files such as `config.json` in `node_modules/backstopjs/config.json`. I would like to suggest to use `os.tmpdir` instead.

My case might be special in that my `node_modules` folder is actually read-only but I really think that `node_modules` should only ever be modified by `npm` and not by random packages storing temporary data there. It does not seem like an idiomatic behavior to me either - I haven't had this issue with any other package so far.

Would there be any objections against storing temporary data in directory below `os.tmpdir()` ?

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.