diff options
author | Caolán McNamara <caolanm@redhat.com> | 2017-09-11 12:13:20 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2017-09-11 15:01:41 +0200 |
commit | 354d460af2f74b74406967da63567e60a1fe782e (patch) | |
tree | eef64478bc944e975a860655297716f0f0da155b /.git-hooks | |
parent | 68e1eb77c9ea2887ba4c113f307c764bc7a3eeef (diff) |
insist in a domain in .ui files
and just check .ui files for those .ui-specific patterns
Change-Id: If7b9c8779f7592db5b22ab96058dd85666ca9bdf
Reviewed-on: https://gerrit.libreoffice.org/42160
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to '.git-hooks')
-rwxr-xr-x | .git-hooks/pre-commit | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/.git-hooks/pre-commit b/.git-hooks/pre-commit index 07569905488b..e35156f012e6 100755 --- a/.git-hooks/pre-commit +++ b/.git-hooks/pre-commit @@ -86,17 +86,20 @@ sub check_whitespaces($) } if (/<property name="use_markup">True<\/property>/) { - bad_line("use font attributes instead of use-markup", $_, $src_limited); + bad_line("use font attributes instead of use-markup", $_, "ui"); } if (/<property name="tooltip_markup"/ ) { - bad_line("use tooltip_text instead of tooltip_markup", $_, $src_limited); + bad_line("use tooltip_text instead of tooltip_markup", $_, "ui"); } if ((/translatable="yes"/) and not(/context=/)) { - bad_line("translatable .ui file line without context", $_, $src_limited); + bad_line("translatable .ui file line without context", $_, "ui"); + } + if ((/<interface/) and not(/domain=/)) + { + bad_line(".ui file without translation domain", $_, "ui"); } - } } if ( $found_bad) |