Skip to content

ui: clean up Parca Context#6215

Merged
yomete merged 3 commits into
mainfrom
external-profiler
Feb 25, 2026
Merged

ui: clean up Parca Context#6215
yomete merged 3 commits into
mainfrom
external-profiler

Conversation

@yomete
Copy link
Copy Markdown
Contributor

@yomete yomete commented Feb 25, 2026

No description provided.

@alwaysmeticulous
Copy link
Copy Markdown

alwaysmeticulous Bot commented Feb 25, 2026

✅ Meticulous spotted 0 visual differences across 394 screens tested: view results.

Meticulous evaluated ~4 hours of user flows against your PR.

Expected differences? Click here. Last updated for commit c964793. This comment will update as new commits are pushed.

Copy link
Copy Markdown
Contributor

@manojVivek manojVivek left a comment

Choose a reason for hiding this comment

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

Just a minor comment, looks good otherwise!

error?: RpcError;
}

export interface ExternalProfilerComponentProps {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Leftover type def after this is moved to the other file?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

thanks for catching that!

@yomete yomete merged commit afb37cc into main Feb 25, 2026
35 checks passed
@yomete yomete deleted the external-profiler branch February 25, 2026 11:30
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.

2 participants