Skip to content

Include OTel classes in baseline profile#2350

Merged
bidetofevil merged 1 commit intomainfrom
hho/baselineprofile-wf
Jul 15, 2025
Merged

Include OTel classes in baseline profile#2350
bidetofevil merged 1 commit intomainfrom
hho/baselineprofile-wf

Conversation

@bidetofevil
Copy link
Copy Markdown
Contributor

@bidetofevil bidetofevil commented Jul 11, 2025

Goal

The generation of the baseline profile now does the appropriate filtering, so the task only needs to copy over and rename the file.

@bidetofevil bidetofevil requested a review from a team as a code owner July 11, 2025 22:31
Copy link
Copy Markdown
Contributor Author

bidetofevil commented Jul 11, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@bidetofevil
Copy link
Copy Markdown
Contributor Author

Updated this to not filter out unwanted rules via grep and just take the entire file as it is filtered now

Copy link
Copy Markdown
Member

@fractalwrench fractalwrench left a comment

Choose a reason for hiding this comment

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

LGTM

@bidetofevil bidetofevil changed the base branch from hho/macrobenchmarks to graphite-base/2350 July 15, 2025 14:39
@bidetofevil bidetofevil force-pushed the hho/baselineprofile-wf branch from c2d234e to 5dddb49 Compare July 15, 2025 14:39
@graphite-app graphite-app bot changed the base branch from graphite-base/2350 to main July 15, 2025 14:40
@bidetofevil bidetofevil force-pushed the hho/baselineprofile-wf branch from 5dddb49 to 054b3c2 Compare July 15, 2025 14:40
@github-actions
Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@bidetofevil bidetofevil merged commit 7fb2a43 into main Jul 15, 2025
14 checks passed
Copy link
Copy Markdown
Contributor Author

Merge activity

@bidetofevil bidetofevil deleted the hho/baselineprofile-wf branch July 15, 2025 14:49
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