-
Notifications
You must be signed in to change notification settings - Fork 8
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
Consider using more featured shard as dependency #1
Comments
I currently have very small experience in Amber, but I'm open to help if needed |
@vladfaust Oh, Thank you! BTW, Have you tried https://github.com/TechMagister/i18n.cr by @TechMagister ? (I just saw the comment in your reddit post 😅 ) |
I haven’t tried the shard you’ve mentioned yet 🙂 |
@faustinoaq what do you think about folding TechMagister's code into Citrine? |
Well, if @TechMagister has no problem with that, It's ok for me 😅 |
I have no problem with that |
@TechMagister Shorter chain of dependencies, and code we can maintain. |
@TechMagister can you merge this PR crimson-knight/i18n.cr#6? |
I think we can enhance this library by integrating a more featured shard like https://github.com/vladfaust/i18n.cr by @vladfaust
@amberframework/contributors WDYT?
The text was updated successfully, but these errors were encountered: