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 po-to-i18n script #7016
Fix po-to-i18n script #7016
Conversation
c13b954
to
1dd1886
Compare
frontend/.eslintignore
Outdated
@@ -7,3 +7,4 @@ public/lib | |||
Godeps | |||
@types | |||
dynamic-demo-plugin | |||
i18n-scripts |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We probably want to keep linting, just use a different eslintrc in the i18n-scripts folder with different rules.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I moved in an eslintrc file; I'm still going through there and removing things that aren't necessary for this folder.
1dd1886
to
e63a1d1
Compare
File structure change broke the script. I fixed that and added better handling for nonexistent directories and bad file names. I also updated .eslintignore so we could remove comments.
e63a1d1
to
07fbc3d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: rebeccaalpert, spadgett The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
I didn't realize recent file structure changes broke the script. I fixed that and added better handling for nonexistent directories and bad file names. I also updated .eslintignore so we could remove "ignore me linter!" comments.