You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There is already an npm script to update snapshot for unit tests: npm run test:update-snapshots.
However, I couldn't find one for Playwright e2e tests.
The readme mentions the command to update the snapshots but it will be nice an "symmetrical" to unit test's if it's available as npm script as well.
The text was updated successfully, but these errors were encountered:
There is already an npm script to update snapshot for unit tests: npm run test:update-snapshots.
However, I couldn't find one for Playwright e2e tests.
I have no objection to us adding more npm scripts.
However I'd lean the other way:
Keep the npm scripts to the bare minimum.
Lean on documentation to encourage deeper understanding of our toolchain.
If we have less scripts / aliases in our package.json it encourages everyone to learn the tooling directly. Scripts can act like a "buffer", a crutch on top of the tooling.
There is already an npm script to update snapshot for unit tests:
npm run test:update-snapshots
.However, I couldn't find one for Playwright e2e tests.
The readme mentions the command to update the snapshots but it will be nice an "symmetrical" to unit test's if it's available as npm script as well.
The text was updated successfully, but these errors were encountered: