Skip to content
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

[playwright] Improve getting started documentation #11094

Merged
merged 2 commits into from
May 5, 2022

Conversation

planger
Copy link
Contributor

@planger planger commented May 2, 2022

What it does

Improves the documentation for getting started with the theia-playwright page object library.

This change requires eclipse-theia/theia-playwright-template#1
And fixes #11015

How to test

Please read examples/playwright/docs/GETTING_STARTED.md and follow the steps in order to evaluate whether this guide will help newcomers to get started easily.

Review checklist

Reminder for reviewers

Requires eclipse-theia/theia-playwright-template#1
Fixes eclipse-theia#11015

Change-Id: I5838de12b7e3e840e354e8230a35c752c9f25dd1
Signed-off-by: Philip Langer <[email protected]>
@vince-fugnitto vince-fugnitto added documentation issues related to documentation playwright issues related to playwright tests labels May 3, 2022

## Adding further tests

Let's write a system test for the Theia text editor as an example:
Let's write add another system test for the Theia text editor as an example:
Copy link
Contributor

Choose a reason for hiding this comment

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

write or add
please remove one of the words

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nice catch! Thanks! Fixed in 4bedc18

Change-Id: I0fa6890334d90831fab6f22e0a169b0b28dca081
@JonasHelming JonasHelming merged commit b9e1949 into eclipse-theia:master May 5, 2022
@JonasHelming JonasHelming added this to the 1.26.0 milestone May 5, 2022
@planger planger deleted the issues/11015 branch February 3, 2023 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation issues related to documentation playwright issues related to playwright tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[playwright] Add documentation for consuming playwright in downstream applications
3 participants