Data-Liberation-Front / csvlint.rb

The gem behind http://csvlint.io
MIT License
283 stars 86 forks source link

Pre commit hook #276

Closed jrottenberg closed 1 year ago

jrottenberg commented 1 year ago

This PR fixes #275

Changes proposed in this pull request:

Floppy commented 1 year ago

Nice one! Could you add a link to https://pre-commit.com in the README? I wasn't aware of the specific tool, so others may also not be...

jrottenberg commented 1 year ago

Thanks a lot @Floppy ! Will you cut a release ? I hinted v1.2.0 the config for the users to start at that version.

Otherwise if they try to use it on main:

❯ pre-commit run -a
[WARNING] The 'rev' field of repo 'https://github.com/Data-Liberation-Front/csvlint.rb' appears to be a mutable reference (moving tag / branch).  Mutable references are never updated after first install and are not supported.  See https://pre-commit.com/#using-the-latest-version-for-a-repository for more details.  Hint: `pre-commit autoupdate` often fixes this.
[INFO] Installing environment for https://github.com/Data-Liberation-Front/csvlint.rb.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
csvlint..................................................................Passed
Floppy commented 1 year ago

Yeah, I'll push one out shortly! Thanks :)

Floppy commented 1 year ago

release done :)

jrottenberg commented 1 year ago

Thanks a lot, we got it on the official list of hooks : https://github.com/pre-commit/pre-commit.com/pull/793

https://pre-commit.com/hooks.html 🎉