Skip to content

docs: fix deprecated Playwright API usage in running-code.md#39403

Merged
dgozman merged 2 commits intomicrosoft:mainfrom
willcoliveira:docs/update-deprecated-playwright-apis
Mar 2, 2026
Merged

docs: fix deprecated Playwright API usage in running-code.md#39403
dgozman merged 2 commits intomicrosoft:mainfrom
willcoliveira:docs/update-deprecated-playwright-apis

Conversation

@willcoliveira
Copy link
Contributor

Summary

Test plan

  • Verify all code examples in running-code.md use current Playwright APIs
  • Confirm no deprecated methods remain (page.click, page.fill, page.waitForSelector)

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@yury-s
Copy link
Member

yury-s commented Feb 27, 2026

Let's resolve the conflicts and we can merge this.

@willcoliveira
Copy link
Contributor Author

Let's resolve the conflicts and we can merge this.

@yury-s , done. Thanks for the heads up.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 2, 2026

Test results for "tests 1"

4 failed
❌ [playwright-test] › playwright.trace.spec.ts:137 › should not throw with trace: on-first-retry and two retries in the same worker @macos-latest-node20
❌ [playwright-test] › reporter-line.spec.ts:193 › merged › should show error context with relative path @macos-latest-node20
❌ [playwright-test] › reporter.spec.ts:251 › created › should not have internal error when steps are finished after timeout @macos-latest-node20
❌ [playwright-test] › reporter.spec.ts:251 › merged › should not have internal error when steps are finished after timeout @macos-latest-node20

2 flaky ⚠️ [chromium-library] › library/trace-viewer.spec.ts:1216 › should display language-specific locators `@chromium-ubuntu-22.04-node20`
⚠️ [playwright-test] › ui-mode-trace.spec.ts:700 › should indicate current test status `@macos-latest-node20`

38657 passed, 841 skipped


Merge workflow run.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 2, 2026

Test results for "MCP"

2 failed
❌ [chrome] › mcp/http.spec.ts:199 › http transport browser lifecycle (persistent) @mcp-macos-15
❌ [chrome] › mcp/launch.spec.ts:116 › isolated context @mcp-macos-15

5131 passed, 171 skipped


Merge workflow run.

@dgozman
Copy link
Contributor

dgozman commented Mar 2, 2026

Thank you for the PR!

@dgozman dgozman merged commit ad06412 into microsoft:main Mar 2, 2026
36 of 38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants