Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: pre-commit hook #513

Merged
merged 1 commit into from
Aug 5, 2024
Merged

feat: pre-commit hook #513

merged 1 commit into from
Aug 5, 2024

Conversation

MarioRodrigues10
Copy link
Member

Description

This pull request introduces a pre-commit hook to ensure code quality before commits are finalized. The hook performs the following checks:

  • Formatting
  • Linting
  • Testing

The setup for this pre-commit hook is handled by running bin/setup, which configures the hook whether you are working in a Docker environment or locally.

Additionally, this PR includes updates made to meet Credo CI requirements, which accounts for the additional changes beyond just the hook setup.

@MarioRodrigues10 MarioRodrigues10 added the enhancement New feature or improvements label Aug 5, 2024
@MarioRodrigues10 MarioRodrigues10 self-assigned this Aug 5, 2024
@ruilopesm ruilopesm requested review from JoaoCoelho2003 and removed request for joao-coelho August 5, 2024 22:38
bin/pre-commit.sh Outdated Show resolved Hide resolved
bin/setup Outdated Show resolved Hide resolved
@ruilopesm ruilopesm linked an issue Aug 5, 2024 that may be closed by this pull request
@MarioRodrigues10 MarioRodrigues10 merged commit 05671d5 into develop Aug 5, 2024
2 checks passed
@MarioRodrigues10 MarioRodrigues10 deleted the mr/pre-hook-test branch August 5, 2024 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add pre-commit hook
3 participants