Skip to content

Comments

feat(create): Implement Cache for CreateInputs::created_address#3218

Merged
rakita merged 2 commits intobluealloy:mainfrom
developeruche:feat/cache-created-addess
Dec 10, 2025
Merged

feat(create): Implement Cache for CreateInputs::created_address#3218
rakita merged 2 commits intobluealloy:mainfrom
developeruche:feat/cache-created-addess

Conversation

@developeruche
Copy link
Contributor

Closes #3217

@codspeed-hq
Copy link

codspeed-hq bot commented Dec 10, 2025

CodSpeed Performance Report

Merging #3218 will not alter performance

Comparing developeruche:feat/cache-created-addess (aeca0f9) with main (9afd166)

Summary

✅ 173 untouched

Copy link
Member

@rakita rakita left a comment

Choose a reason for hiding this comment

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

cool
lgtm

@rakita rakita merged commit 58c49fb into bluealloy:main Dec 10, 2025
31 checks passed
@github-actions github-actions bot mentioned this pull request Dec 9, 2025
@DaniPopes
Copy link
Collaborator

the fields shouldnt be pub cuz it invalidates the cache but its not reset

@developeruche
Copy link
Contributor Author

the fields shouldnt be pub cuz it invalidates the cache, but it's not reset

Got you... I would expose a getter method for the fields and make them private.

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.

Cache CreateInputs::created_address

3 participants