Apply suggestions from code review
Co-authored-by: Yury Semikhatsky <yurys@chromium.org> Signed-off-by: Simon Knott <info@simonknott.de>
This commit is contained in:
parent
9a8aaa4bac
commit
d786a27ed5
|
|
@ -261,7 +261,7 @@ npx playwright --version
|
||||||
|
|
||||||
Setup CI/CD and run your tests frequently. The more often you run your tests the better. Ideally you should run your tests on each commit and pull request. Playwright comes with a [GitHub actions workflow](/ci-intro.md) so that tests will run on CI for you with no setup required. Playwright can also be setup on the [CI environment](/ci.md) of your choice.
|
Setup CI/CD and run your tests frequently. The more often you run your tests the better. Ideally you should run your tests on each commit and pull request. Playwright comes with a [GitHub actions workflow](/ci-intro.md) so that tests will run on CI for you with no setup required. Playwright can also be setup on the [CI environment](/ci.md) of your choice.
|
||||||
|
|
||||||
Use Linux when running your tests on CI as it is cheaper. Developers can use whatever environment when running locally but use linux on CI. Set up [Sharding](./test-sharding.md) to make CI faster.
|
Use Linux when running your tests on CI as it is cheaper. Developers can use whatever environment when running locally but use linux on CI. Consider setting up [Sharding](./test-sharding.md) to make CI faster.
|
||||||
|
|
||||||
### Lint your tests
|
### Lint your tests
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -61,9 +61,9 @@ export default defineConfig({
|
||||||
|
|
||||||
## CI configurations
|
## CI configurations
|
||||||
|
|
||||||
### GitHub Actions
|
The [Command line tools](./browsers#install-system-dependencies) can be used to install all operating system dependencies in CI.
|
||||||
|
|
||||||
The [Command line tools](./browsers#install-system-dependencies) can be used to install all operating system dependencies on GitHub Actions.
|
### GitHub Actions
|
||||||
|
|
||||||
#### On push/pull_request
|
#### On push/pull_request
|
||||||
* langs: js
|
* langs: js
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue