Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: importAnalysisBuild #19723

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

privatenumber
Copy link
Contributor

Description

Pulling out some minor refactors from #19722

@privatenumber privatenumber force-pushed the refactor-import-analysis-build branch from 154314e to 4098d6f Compare March 26, 2025 04:57
@privatenumber privatenumber marked this pull request as ready for review March 26, 2025 05:04
@privatenumber privatenumber force-pushed the refactor-import-analysis-build branch 3 times, most recently from e5912cc to 0d85f7d Compare March 26, 2025 06:08
@sapphi-red sapphi-red added the p1-chore Doesn't change code behavior (priority) label Mar 27, 2025
@privatenumber privatenumber force-pushed the refactor-import-analysis-build branch 6 times, most recently from 309a0f4 to aaa2273 Compare March 27, 2025 13:00
@privatenumber privatenumber force-pushed the refactor-import-analysis-build branch 8 times, most recently from 62ce4aa to 0325471 Compare March 28, 2025 08:24
sapphi-red
sapphi-red previously approved these changes Mar 28, 2025
@privatenumber privatenumber force-pushed the refactor-import-analysis-build branch 2 times, most recently from 0958fd6 to 66cd77e Compare March 28, 2025 10:33
@privatenumber
Copy link
Contributor Author

With this refactor, I also have a follow up perf optimization to de-duplicate the chunk registry:
#19757

@privatenumber privatenumber force-pushed the refactor-import-analysis-build branch from 66cd77e to 2496007 Compare March 31, 2025 03:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p1-chore Doesn't change code behavior (priority)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants