Skip to content

gh-145306: Fix browser open after empty export#150017

Merged
pablogsal merged 2 commits into
python:mainfrom
lkollar:tachyon/fix-browser-open
Jun 27, 2026
Merged

gh-145306: Fix browser open after empty export#150017
pablogsal merged 2 commits into
python:mainfrom
lkollar:tachyon/fix-browser-open

Conversation

@ivonastojanovic

Copy link
Copy Markdown
Contributor

Make sampling exporters report whether output was generated, and only open HTML reports in the browser after a successful export.

Comment thread Lib/profiling/sampling/cli.py Outdated
@picnixz picnixz changed the title gh-145306 Fix browser open after empty export gh-145306: Fix browser open after empty export May 19, 2026
lkollar added 2 commits May 21, 2026 16:49
Make sampling exporters report whether output was generated, and only
open HTML reports in the browser after a successful export.
@lkollar lkollar force-pushed the tachyon/fix-browser-open branch from e98a58b to 4334b0f Compare May 21, 2026 15:51
@pablogsal pablogsal merged commit 860f8a5 into python:main Jun 27, 2026
53 checks passed
@pablogsal pablogsal added awaiting merge needs backport to 3.15 pre-release feature fixes, bugs and security fixes labels Jun 27, 2026
@miss-islington-app

Copy link
Copy Markdown

Thanks @ivonastojanovic for the PR, and @pablogsal for merging it 🌮🎉.. I'm working now to backport this PR to: 3.15.
🐍🍒⛏🤖

@bedevere-app

bedevere-app Bot commented Jun 27, 2026

Copy link
Copy Markdown

GH-152447 is a backport of this pull request to the 3.15 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.15 pre-release feature fixes, bugs and security fixes label Jun 27, 2026
pablogsal pushed a commit that referenced this pull request Jun 27, 2026
…52447)

gh-145306: Fix browser open after empty export (GH-150017)
(cherry picked from commit 860f8a5)

Co-authored-by: ivonastojanovic <80911834+ivonastojanovic@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants