-
Notifications
You must be signed in to change notification settings - Fork 33
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
Verify absence of fuzzy translations #596
Conversation
Rebased onto current develop. |
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.
It looks like there are too many files in this PR.
Yeah, I'm cleaning it up. Let's see if we actually need the PO files in the MANIFEST. |
It finally passes Travis, and I've cleaned up the accidentally included changes. @matsduf, @vlevigneron Please review again. |
} | ||
|
||
subtest "no fuzzy marks" => sub { | ||
my @fuzzy_po_files = qw( da.po fr.po sv.po ); |
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.
Is this a list of po files that must be fuzzy free or may have "fuzzy"? What about en.po?
Why not fetch from share/ instead to make it catch future languages automatically?
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.
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.
Since the "fuzzy" has been removed from sv and da I do not see any reason to keep those on a white list (or black list), but OK.
This adds a new unit test ensuring that we don't accept fuzzy markings in PO files into the develop branch.
Three out of four PO files do contain fuzzy entries, so they have been added to a whitelist to be opted out of this check. For each one there is an issue to clean up the PO file:
sv.po
fr.po
da.po
There is also an issue (#597) to clean up the whitelist mechanism once all PO files have been removed from the whitelist.