Skip to content

Commit fb55abb

Browse files
authored
fix up pre-commit for xliff files (#19426)
### Summary of the issue: Adding new languages to NVDA from Crowdin causes pre-commit to fail, as it commits large xliff files. Failed job: https://github.com/nvaccess/nvda/actions/runs/20842477976/job/59879416174 ### Description of user facing changes: None ### Description of developer facing changes: We can add new languages to NVDA via Crowdin ### Description of development approach: update pre-commit ### Testing strategy: Tested trying to commit a large xliff file locally ### Known issues with pull request: n/a
1 parent 84e14ee commit fb55abb

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

.pre-commit-config.yaml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,10 @@ repos:
4141
# Checks that large files have not been added. Default cut-off for "large" files is 500kb.
4242
- id: check-added-large-files
4343
# POFiles and TTF fonts can't be made smaller
44+
# XLIFF files can be moved here from exclude when pre-commit/identify#558 is released.
4445
exclude_types: ["pofile", "ttf"]
45-
# Same applies for NVDA dictionary (.dic) files and Spline Font Database (.SFD) files, but these aren't recognised by the Identify library.
46-
exclude: "\\.(dic|sfd)$"
46+
# Same applies for XLIFF, NVDA dictionary (.dic) files and Spline Font Database (.SFD) files, but these aren't recognised by the Identify library.
47+
exclude: "\\.(dic|sfd|xliff)$"
4748
# Checks python syntax
4849
- id: check-ast
4950
# Checks for filenames that will conflict on case insensitive filesystems (the majority of Windows filesystems, most of the time)

0 commit comments

Comments
 (0)