Pull-Request-Community / pull-request-community-website

Pull request community
https://pullrequest.co.il/
MIT License
56 stars 101 forks source link

ci: fix e2e tests by updating Playwright #291

Closed OmriBarZik closed 1 year ago

OmriBarZik commented 1 year ago

WHATS NEW

@MichalPorag @lirantal @abrl91

Eslint + json by #290

github-actions[bot] commented 1 year ago

Preview on netlify: https://deploy-preview-291--pull-request.netlify.app/

netlify[bot] commented 1 year ago

Deploy Preview for agitated-hoover-61371e ready!

Built without sensitive environment variables

Name Link
Latest commit 13daae71e853d262cceeea5eb30ddd92ab52398d
Latest deploy log https://app.netlify.com/sites/agitated-hoover-61371e/deploys/64dbdce94be86c0008fd4c75
Deploy Preview https://deploy-preview-291--agitated-hoover-61371e.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

OmriBarZik commented 1 year ago

Looks like it's failing on code formatting issues with one of the person entries, and the other CI job fails too - perhaps needs a dependency update too.

@lirantal created a fix #290

lirantal commented 1 year ago

Saw it, let's merge that one and rebase here to get the fix and work out the e2e issue.