Skip to content

docs: use relative links for internal packages and configs in CONTRIBUTING.md - #42416

Closed
Jack Thomas (loulanyue) wants to merge 1 commit into
microsoft:mainfrom
loulanyue:docs/use-relative-links-contributing
Closed

docs: use relative links for internal packages and configs in CONTRIBUTING.md#42416
Jack Thomas (loulanyue) wants to merge 1 commit into
microsoft:mainfrom
loulanyue:docs/use-relative-links-contributing

Conversation

@loulanyue

Copy link
Copy Markdown

Summary

Replaces hardcoded https://github.com/microsoft/playwright/blob/main/... links in CONTRIBUTING.md with relative markdown links to improve navigability across forks, branches, and local browsing.

Comment thread CONTRIBUTING.md

Playwright requires a test for almost any new or modified functionality. An exception would be a pure refactoring, but chances are you are doing more than that.

There are multiple [test suites](https://github.com/microsoft/playwright/blob/main/tests) in Playwright that will be executed on the CI. The two most important that you need to run locally are:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also this?

Comment thread CONTRIBUTING.md

**Building documentation site**

The [playwright.dev](https://playwright.dev/) documentation site lives in a separate repository, and documentation from [`docs/src`](https://github.com/microsoft/playwright/blob/main/docs/src) is frequently rolled there.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also this?

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "MCP"

8259 passed, 1361 skipped


Merge workflow run.

@github-actions

Copy link
Copy Markdown
Contributor

Test results for "tests 1"

2 flaky ⚠️ [firefox-library] › library/heap.spec.ts:223 › should not leak workers `@firefox-ubuntu-22.04-node20`
⚠️ [firefox-page] › page/page-goto.spec.ts:90 › should work with Cross-Origin-Opener-Policy `@firefox-ubuntu-22.04-node20`

51266 passed, 1237 skipped


Merge workflow run.

@loulanyue Jack Thomas (loulanyue) closed this by deleting the head repository Aug 27, 2026
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.

2 participants