Doesn't just running the pre-commit hooks locally (black and ruff) ensure this consistency ?

Doesn't just running the pre-commit hooks locally (black and ruff) ensure this consistency ?

If that is the case, how did your pr results so many different files?

I ran the pre-commit hooks on all files before committing so the changes are white spaces and formatting that the linter enforced - I only modified one file in my PR

there is no black and ruff config in the pyproject.toml. Maybe your configuration and the others somehow end up being different. Ill add a simple version and you can rebase on this, and try to use the format and test if those files can be set back

https://github.com/SylphAI-Inc/AdalFlow/pull/274/files

Feel free to review, merge and test it on your side

You can use make setup and make format, make lint etc

Yes I assumed that the fact that they were part of the .pre-commit-config.yaml means they are enforced. I will have a look

I ran the pre-commit hooks suggested on all files, made the changes 3196631. Maybe this can be merged and then the styling will be consistent ? @liyin2015

• Three files need further improvement so that we can enforce pre-commit check on formatting and linting. (1) Makefile (2) pyproject.toml on the relevant confi (3).pre-commit-config.yaml

Proposed Changes

1. Update the formatting tools configuration:

   • Add explicit Black and Ruff configurations to root pyproject.toml
   • Configure both tools to use the same line length (88) and Python version
   • Set consistent exclude patterns for docs and non-Python files

2. Improve the Makefile commands:

   • Make format and lint commands use the pyproject.toml config
   • Add Ruff format alongside Black for complete formatting coverage

3. Update .pre-commit-config.yaml:

   • Point all hooks to use the pyproject.toml config
   • Add Ruff format hook
   • Maintain consistent exclude patterns

Implementation steps:

1. Add proposed configurations
2. Run make setup and make format on all files
3. Verify with make lint

This should resolve the inconsistent formatting issues we're seeing across PRs. I can implement these changes if everyone agrees with this approach.

Proposed Changes

1. Update the formatting tools configuration:

   • Add explicit Black and Ruff configurations to root pyproject.toml
   • Configure both tools to use the same line length (88) and Python version
   • Set consistent exclude patterns for docs and non-Python files

2. Improve the Makefile commands:

   • Make format and lint commands use the pyproject.toml config
   • Add Ruff format alongside Black for complete formatting coverage

3. Update .pre-commit-config.yaml:

   • Point all hooks to use the pyproject.toml config
   • Add Ruff format hook
   • Maintain consistent exclude patterns

Implementation steps:

1. Add proposed configurations
2. Run make setup and make format on all files
3. Verify with make lint

This should resolve the inconsistent formatting issues we're seeing across PRs. I can implement these changes if everyone agrees with this approach.

Yea I think this works you can do a PR again if you wish. Thanks