Skip to content

fix: Build the ChromeNotFoundError message as one string - #314

Open
Blizzeq wants to merge 2 commits into
plotly:mainfrom
Blizzeq:fix/chrome-not-found-message
Open

Blizzeq wants to merge 2 commits into
plotly:mainfrom
Blizzeq:fix/chrome-not-found-message

Conversation

@Blizzeq

@Blizzeq Blizzeq commented Sep 18, 2026

Copy link
Copy Markdown

Description

Chromium passed the "browser not found" message to ChromeNotFoundError as two arguments, because of a stray comma between the f-strings. So str(e) printed a tuple:

('Browser not found. ... Local copy ignored: False. ', 'Path calculated:: /nonexistent/chrome.')

The message is now one string, and the doubled colon after "Path calculated" is gone. A test in test_chromium.py checks both. It fails on main.

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.

1 participant