Skip to content

node: Add --no-playwright-browsers flag - #553

Merged
bbhtt merged 1 commit into
flatpak:masterfrom
threema-sebastian:add---no-playwright-browsers
Oct 22, 2026
Merged

bbhtt merged 1 commit into
flatpak:masterfrom
threema-sebastian:add---no-playwright-browsers

Conversation

@threema-sebastian

Copy link
Copy Markdown
Contributor

To avoid fetching the playwright browsers in cases where we don't need them in the final flatpack sandbox.

@bbhtt

bbhtt commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

can you add a test too?

@threema-sebastian

threema-sebastian commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor Author

sorry wasn't able to get back to this sooner, will provide tests next week

@threema-sebastian
threema-sebastian force-pushed the add---no-playwright-browsers branch from 868e45d to f9c5f7b Compare October 19, 2026 10:19
@threema-sebastian

Copy link
Copy Markdown
Contributor Author

can you add a test too?

Okay added a test that asserts that no requests went out for playwright if the flag has been passed

@bbhtt

bbhtt commented Oct 22, 2026

Copy link
Copy Markdown
Collaborator

CI is failing.

@threema-sebastian
threema-sebastian force-pushed the add---no-playwright-browsers branch from f9c5f7b to c188e47 Compare October 22, 2026 13:27
To avoid fetching the playwright browsers in cases where we don't need
them in the final flatpack.
@threema-sebastian
threema-sebastian force-pushed the add---no-playwright-browsers branch from c188e47 to 270e131 Compare October 22, 2026 13:28
@threema-sebastian

Copy link
Copy Markdown
Contributor Author

CI is failing.

Fixed the formatting issue and rebased the branch

@bbhtt
bbhtt merged commit 3d6ff66 into flatpak:master Oct 22, 2026
4 checks passed
@threema-sebastian
threema-sebastian deleted the add---no-playwright-browsers branch October 22, 2026 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants