Skip to content

Conversation

@sunny
Copy link
Member

@sunny sunny commented Dec 9, 2025

Closes #1172

@sunny sunny requested a review from yukideluxe December 9, 2025 13:16
@sunny sunny force-pushed the fix-currency-table-not-initialized-on-register branch from db523ba to 74797be Compare December 9, 2025 13:20
@sunny sunny force-pushed the fix-currency-table-not-initialized-on-register branch from 74797be to 210923a Compare December 9, 2025 13:21
Copy link
Member

@yukideluxe yukideluxe left a comment

Choose a reason for hiding this comment

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

Ouch! I actually think this is happening since #1143 because the @table is not loaded until a currency is initialized 😳 😳 😳 😳 😳 😳 😳

I would not be against releasing a new version! What do you think, @sunny!?

@sunny
Copy link
Member Author

sunny commented Dec 9, 2025

Good call about #1143, it makes sense and a git bisect confirms it. Let’s go for a v7.0.2 🚀

@sunny sunny merged commit da05a31 into RubyMoney:main Dec 9, 2025
7 checks passed
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.

Money::Currency.table not initialized if Money::Currency.register called first

2 participants