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

Bump i18n dependency #280

Merged
merged 2 commits into from
Sep 25, 2018
Merged

Bump i18n dependency #280

merged 2 commits into from
Sep 25, 2018

Conversation

shioyama
Copy link
Owner

Fixes #278

@shioyama
Copy link
Owner Author

@omitter so it does seem there is a small regression with I18n 1.1, two specs related to fallbacks are failing. I'll have to look into it, but if you want to have a look that would help speed things up.

@shioyama
Copy link
Owner Author

@shioyama
Copy link
Owner Author

I guess it's probably ruby-i18n/i18n#415

@omitter
Copy link
Contributor

omitter commented Sep 21, 2018

Checking the linked commit, it looks like they simply deleted the fall back tests (here). But I guess it's not a good idea to do this here too.

@shioyama shioyama force-pushed the bump_i18n_dependency branch from f44caf5 to f5b1994 Compare September 21, 2018 08:23
@shioyama shioyama force-pushed the bump_i18n_dependency branch from f5b1994 to 3bc22dd Compare September 23, 2018 12:21
@shioyama shioyama merged commit 3bc22dd into master Sep 25, 2018
@shioyama shioyama deleted the bump_i18n_dependency branch September 25, 2018 07:25
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