Skip to content

Comments

Revert "Split precompile constants out of individual modules #2081"#2101

Closed
mattsse wants to merge 1 commit intomainfrom
revert-2085-Split-Precompile-Constants-out-of-Individual-Modules-#2081
Closed

Revert "Split precompile constants out of individual modules #2081"#2101
mattsse wants to merge 1 commit intomainfrom
revert-2085-Split-Precompile-Constants-out-of-Individual-Modules-#2081

Conversation

@mattsse
Copy link
Collaborator

@mattsse mattsse commented Feb 20, 2025

Reverts #2085

this change introduced a bug, we should start over here and do it right

@codspeed-hq
Copy link

codspeed-hq bot commented Feb 20, 2025

CodSpeed Performance Report

Merging #2101 will not alter performance

Comparing revert-2085-Split-Precompile-Constants-out-of-Individual-Modules-#2081 (a26f7e6) with main (02c9b67)

Summary

✅ 8 untouched benchmarks

@rakita
Copy link
Member

rakita commented Feb 20, 2025

Will close this as bugs have been fixed here: #2102 and CI for prague is enabled

@rakita rakita closed this Feb 20, 2025
@rakita rakita deleted the revert-2085-Split-Precompile-Constants-out-of-Individual-Modules-#2081 branch May 8, 2025 13:36
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