Skip to content

Commit

Permalink
Adjusts pre-commit to work on Windows (#2958)
Browse files Browse the repository at this point in the history
# Description and Motivation
<!--- bulleted, high level items. use keywords (eg "closes #144" or
"fixes #4323") -->

For Windows, it worked when i specified the `bash` command rather than
just using `./` to invoke the sh script

fixes #2954 

## Has this been tested? How?

working on my Windows VM and on my Mac. I did also add some entries to
the PATH environmental variables on windows, not sure if that was needed
to get this to work

## Screenshots (if appropriate)

<img width="734" alt="Screenshot 2024-03-05 at 9 22 31 AM"
src="https://github.com/SatcherInstitute/health-equity-tracker/assets/41567007/5fca3dfa-3615-40f7-9a44-b8ebcbc1cfc5">


## Types of changes

(leave all that apply)

- Bug fix

## New frontend preview link is below in the Netlify comment 😎


Co-authored-by: James Demlow <[email protected]>
  • Loading branch information
benhammondmusic and JDemlow authored Mar 5, 2024
1 parent 15f7d8a commit dab1abd
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ repos:
hooks:
- id: check-file-count
name: 🗄️ check + warn if 100+ files in commit
entry: ./scripts/.util/warn_many_files_per_commit.sh
entry: bash -c scripts/.util/warn_many_files_per_commit.sh
language: system
types: [text]
pass_filenames: false

0 comments on commit dab1abd

Please sign in to comment.