w3c / aria-at

Assistive Technology ARIA Experience Assessment
https://aria-at.netlify.app
Other
154 stars 28 forks source link

Add `*.csv` to .gitattributes and prevent BOM #1112

Open howard-e opened 3 months ago

howard-e commented 3 months ago

Preview Tests

Continued discussion from:

howard-e commented 3 months ago

https://github.com/w3c/aria-at/pull/1108#issuecomment-2296957598

You're right, it looks like there isn't anything native to git that will clean out the BOM. The encoding is set in .editorconfig, but I'm not sure how that would be enforced

@nschonni what do you think of using a git hook in combination with this?

nschonni commented 3 months ago

Yeah, something like https://www.npmjs.com/package/strip-bom-cli with husky + CI doublechecking would probably work