-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add playwright #1119
Add playwright #1119
Conversation
🦙 MegaLinter status: ✅ SUCCESS
See errors details in artifact MegaLinter reports on CI Job page |
c08038c
to
a98c506
Compare
.github/workflows/build-test.yaml
Outdated
- run: yarn install --frozen-lockfile | ||
working-directory: playwright | ||
- run: npx playwright install --with-deps | ||
working-directory: playwright |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note, after yarn install playwright needs more steps to start working https://playwright.dev/docs/cli#install-system-dependencies
{ | ||
"scripts": { | ||
"test": "playwright test" | ||
}, | ||
"devDependencies": { | ||
"@playwright/test": "1.27.1" | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is isolated in a separate folder for now
a98c506
to
e0fd8a8
Compare
9fc459f
to
d657c49
Compare
d657c49
to
81d9940
Compare
Codecov Report
@@ Coverage Diff @@
## master #1119 +/- ##
==========================================
+ Coverage 83.68% 83.90% +0.22%
==========================================
Files 121 121
Lines 2249 2249
Branches 539 539
==========================================
+ Hits 1882 1887 +5
+ Misses 367 362 -5
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Merging this despite failed CI, because it accurately shows that production build on master is broken |
Part of #975
Part of #1061