Skip to content

Adding pre-commit hooks for black and flake8#117

Closed
rodrigobr-msft wants to merge 2 commits intomainfrom
users/robrandao/pre-commit-hooks
Closed

Adding pre-commit hooks for black and flake8#117
rodrigobr-msft wants to merge 2 commits intomainfrom
users/robrandao/pre-commit-hooks

Conversation

@rodrigobr-msft
Copy link
Contributor

This pull request introduces pre-commit hooks and updates the development setup to streamline code quality checks and environment preparation. The main changes are the addition of pre-commit configuration, updating development dependencies, and enhancing the PowerShell setup script.

Pre-commit and development tooling:

  • Added a .pre-commit-config.yaml file to configure black for code formatting and flake8 for linting, ensuring code quality standards are enforced before commits.
  • Updated dev_dependencies.txt to include the pre-commit package, making it available for local development environments.

Development environment setup:

  • Enhanced dev_setup.ps1 to automate virtual environment creation, install editable local libraries, install development dependencies, and set up pre-commit hooks for all contributors.

@rodrigobr-msft
Copy link
Contributor Author

I like Chris' approach of adding documentation better over having a script.

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.

1 participant

Comments