-
-
Notifications
You must be signed in to change notification settings - Fork 81
Lint translation files #382
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
Comments
Will these lint issues block the doc buildings? |
The lint issues do not block anything, they have been around for a while ;-) They sometimes cause issues with how the translation renders, other times they are just quality issues. At pl, we keep the errors at 0. |
build warnings are fixed now but some lint warnings are hard to locate like "clones/rebased_translations/python/python-docs-zh-cn/reference/compound_stmts.po:2642: trailing whitespace (trailing-whitespace)", seems the ref number is not correct. |
The warnings on the dashboard may be slightly out of date. transifex-automations has test building and linting workflows, they are simple to add, just the branch matrix's need to be updated, I highly suggest you add them. You can combine the two my merging the jobs. |
Uh oh!
There was an error while loading. Please reload this page.
There are currently 342 lint warnings when polint is ran, there are also a few build warnings.
transifex-automations has test building and linting workflows, they are simple to add, just the branch matrix's need to be updated.
The text was updated successfully, but these errors were encountered: