Skip to content

Handle the case when the Playwright config defines projects#256

Merged
jtpio merged 2 commits intojupyterlab:mainfrom
jtpio:fix-browser
Jun 25, 2025
Merged

Handle the case when the Playwright config defines projects#256
jtpio merged 2 commits intojupyterlab:mainfrom
jtpio:fix-browser

Conversation

@jtpio
Copy link
Member

@jtpio jtpio commented Jun 24, 2025

This should help fix jupyterlab/jupyterlab#17389

Follow-up to #183.

To keep things simple, specify two distinct steps:

  • when browser is specified as input
  • when browser is not specified

@jtpio jtpio requested a review from krassowski June 24, 2025 19:05
@jtpio jtpio added the bug Something isn't working label Jun 24, 2025
Copy link
Member

@krassowski krassowski left a comment

Choose a reason for hiding this comment

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

Let's try it!

@jtpio
Copy link
Member Author

jtpio commented Jun 25, 2025

Cool, thanks for the review!

@jtpio jtpio merged commit 9f67d26 into jupyterlab:main Jun 25, 2025
26 of 27 checks passed
@jtpio
Copy link
Member Author

jtpio commented Jun 25, 2025

Starting the 0.28.1 release now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Updating documentation snapshot fails

2 participants