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

Single quotes don't need to be escaped here. #2227

Merged
merged 1 commit into from
May 13, 2019
Merged

Single quotes don't need to be escaped here. #2227

merged 1 commit into from
May 13, 2019

Conversation

srbaker
Copy link
Contributor

@srbaker srbaker commented May 13, 2019

This was preventing the app from loading, weirdly.

The error I was getting (in all places where I've made changes) looked like this: I18n::InvalidLocaleData: can not load translations from /Users/srbaker/Projects/TracksApp/tracks/config/locales/en.yml: #<Psych::SyntaxError: (/Users/srbaker/Projects/TracksApp/tracks/config/locales/en.yml): found unknown escape character while parsing a quoted scalar at line 585 column 31>

I don't know why I'm getting this and others aren't: I suspect I'm on a newer libyaml that is more strict.

I've confirmed this with the YAML 1.2 spec, and a few other projects around the web, such as: faker-ruby/faker#1305

This was preventing the app from loading, weirdly.
@srbaker
Copy link
Contributor Author

srbaker commented May 13, 2019

With this one merged, I'll be able to get on with the real work: I'm going to take #2221 and just replace it with Peek.

@mattr-
Copy link
Member

mattr- commented May 13, 2019

LOL, Ruby, why? 🤣

@mattr- mattr- merged commit ec76096 into TracksApp:master May 13, 2019
@srbaker srbaker deleted the fix-invalid-locale-data branch May 15, 2019 20:21
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