Skip to content
This repository has been archived by the owner on May 22, 2019. It is now read-only.

lintspaces is run on binary files #111

Open
ellispritchard opened this issue Sep 5, 2016 · 3 comments
Open

lintspaces is run on binary files #111

ellispritchard opened this issue Sep 5, 2016 · 3 comments

Comments

@ellispritchard
Copy link

I've added a .png file to my repo, for an architecture diagram.

Unfortunately the _verify_lintspaces target tries to run lintspaces on every git recorded file, including the PNG file, which obviously breaks the build.

lintspaces should only run on text files, or restrict to known extensions, rather than on the entire output of git ls-files -z $1 | tr '\0' '\n' | xargs -I {} find {} ! -type l

@matthew-andrews
Copy link
Contributor

hmm, i think the command to execute lintspaces is correct but instead we just need to tweak the .editorconfig file. Are you using the default .editorconfig file or you own version?

Suspect it's this line in the .editorconfig file causing the pain:-
https://github.com/Financial-Times/n-makefile/blob/master/config/.editorconfig#L3

@ellispritchard
Copy link
Author

The default from make updatetools

@matthew-andrews
Copy link
Contributor

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants