Skip to content

Conversation

@cclauss
Copy link
Contributor

@cclauss cclauss commented Nov 26, 2025

Learn more about this config here: https://pre-commit.com

To enable these pre-commit hooks run:

  1. pipx install pre-commit or brew install pre-commit
  2. In the project root directory, run pre-commit install

Includes the codespell changes in #33, but excluding the minified file dist/index.js.

% codespell --ignore-words-list=nd --skip="./dist/index.js"

./tests/fixtures/solr_configs/9.x.x/schema.xml:433: generaly ==> generally
./tests/fixtures/solr_configs/8.x.x/schema.xml:427: generaly ==> generally

% codespell --ignore-words-list=nd --skip="./dist/index.js" --write-changes

@cclauss cclauss force-pushed the pre-commit branch 2 times, most recently from ef37928 to ef36039 Compare November 26, 2025 09:16
@DhavalGojiya
Copy link
Owner

DhavalGojiya commented Nov 26, 2025

fix end of files.........................................................Failed
- hook id: end-of-file-fixer
- exit code: 1
- files were modified by this hook

Fixing dist/scripts/setup-solr.sh

Many hooks will try to change content of dist/ directory, If it doesn't have the expected newline or whitespace.

@DhavalGojiya
Copy link
Owner

@cclauss Rebase

@cclauss
Copy link
Contributor Author

cclauss commented Nov 26, 2025

✅ Rebased.

@DhavalGojiya
Copy link
Owner

Thank you for your hard work @cclauss

@DhavalGojiya DhavalGojiya merged commit ec5106e into DhavalGojiya:main Nov 26, 2025
4 checks passed
@cclauss cclauss deleted the pre-commit branch November 26, 2025 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants