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

Fix formatting and linting. #8

Merged
merged 3 commits into from
Mar 12, 2019
Merged

Fix formatting and linting. #8

merged 3 commits into from
Mar 12, 2019

Conversation

Serabe
Copy link
Contributor

@Serabe Serabe commented Mar 12, 2019

In first place, let me apologize for too many changes in one PR. I needed to fix the Finnish formatting
but linting got into my editor's way so this is the complete list of
changes:

  • Fix formatting for a few languages. Some languages include a dot in
    the short format and in the data we get that dot surrounded by single
    quotes. This is the main reason for this PR.
  • Make the project pass its linting rules.
  • Make CI check linting.
  • Fix path in tslint.json to properly exclude test/locale-data.ts
  • Add test/locale-data.ts to .prettierignore
  • Fix the versions of Node being used in Travis to prevent jest error in Node version over 11.11 facebook/create-react-app#6591 (though it is a create-react-app issue it is related to Jest not working on Node 11.11.0)

Sergio Arbeo added 3 commits March 12, 2019 10:25
In first place, let me apologize. I needed to fix the Finnish formatting
but linting got into my editor's way so this is the complete list of
changes:

- Fix formatting for a few languages. Some languages include a dot in
the short format and in the data we get that dot surrounded by single
quotes. This is the main reason for this PR.
- Make the project pass its linting rules.
- Make CI check linting.
- Fix path in tslint.json to properly exclude test/locale-data.ts
- Add test/locale-data.ts to .prettierignore
Jest is having a problem with latest unstable node (11.11.0) facebook/create-react-app#6591 so trying to set specific supported versions to fix travis
Copy link
Owner

@snewcomer snewcomer left a comment

Choose a reason for hiding this comment

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

Woah that is crazy they have that ending! Thank you so much Sergio!

@snewcomer snewcomer added bug Something isn't working enhancement New feature or request labels Mar 12, 2019
@snewcomer snewcomer merged commit d748af3 into snewcomer:master Mar 12, 2019
@Serabe Serabe deleted the update-formatting branch March 15, 2019 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants