Closed tgreenx closed 2 months ago
This PR updates to the latest playwright version, and a cautionary message in the docs/Testing.md file due to recent events.
docs/Testing.md
It also updates the minimum Node-JS version in the CI (according to https://github.com/zonemaster/zonemaster/blob/master/docs/internal/distrib-testing/Ubuntu-Node.js-build-environment.md).
Tests should pass.
Should package-lock.json be included?
package-lock.json
Yes I believe so. And at least it seems to be the case in many recent commits that changed the package.json file.
package.json
Purpose
This PR updates to the latest playwright version, and a cautionary message in the
docs/Testing.md
file due to recent events.It also updates the minimum Node-JS version in the CI (according to https://github.com/zonemaster/zonemaster/blob/master/docs/internal/distrib-testing/Ubuntu-Node.js-build-environment.md).
How to test this PR
Tests should pass.