Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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

Open
StanFromIreland opened this issue May 18, 2025 · 4 comments
Open

Lint translation files #382

StanFromIreland opened this issue May 18, 2025 · 4 comments

Comments

@StanFromIreland
Copy link

StanFromIreland commented May 18, 2025

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.

@Dumeng
Copy link
Collaborator

Dumeng commented May 18, 2025

Will these lint issues block the doc buildings?

@StanFromIreland
Copy link
Author

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.

@silkriver
Copy link

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.

@StanFromIreland
Copy link
Author

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.

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

No branches or pull requests

3 participants