docs: note Chromium installation before browser verification - #536
wufangyong973 wants to merge 1 commit into
Conversation
|
@wufangyong973 is attempting to deploy a commit to the electrohyun's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Repository: electrohyun/smocket/.coderabbit.yaml Review profile: CHILL Plan: Advanced Run ID:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Thank you for the contribution! The changes address the issue as intended. But, I'd like to keep the remaining good first issues available for other new contributors, so I'll close this PR. I'm happy to merge #534 once you've addressed the review feedback and the required checks pass. |
#522
Both example READMEs offer a browser verification step, but neither said Playwright needs Chromium installed first — CI already does this before running the checks. Added the suggested one-line setup note immediately before the browser verification command block in
examples/drawing-game/README.mdandexamples/shared-worker-lobby/README.md. The Node-only comparison path is left untouched, as the issue notes it does not need Chromium.Docs-only change.
Closes #522.