Closed Bug 1423543 Opened 3 years ago Closed 11 months ago
Fluent whitelist hook should exclude test files
In bug 1422487, we triggered the fluent whitelist hook on the example migration files. gandalf, stas, should we try to reduce the hook to just files that are part of our actual localizations? That would exclude test files, but also files where developers aren't yet ready to expose their files to localization. They could iterate faster, and we'd only require review once they're moving the file to l10n?
Would that mean testing if the file's path contains `/localization/`? Or that it doesn't contain `/test/`? Both?
Flags: needinfo?(gandalf) → needinfo?(l10n)
probably something like path.match(/browser|toolkit|...|/locales/en-US/) ... I'm totally screwing up regex syntax of various languages here.
Component: Localization → Internationalization
Priority: -- → P5
Status: NEW → RESOLVED
Closed: 11 months ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.