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.

Sign up for GitHub

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

Jump to bottom

git hooks should not be mandatory #1

Open
7 tasks
thlmenezes opened this issue Sep 30, 2022 · 0 comments
Open
7 tasks

git hooks should not be mandatory #1

thlmenezes opened this issue Sep 30, 2022 · 0 comments

Comments

@thlmenezes
Copy link
Contributor

thlmenezes commented Sep 30, 2022 •
edited by guico3lho
Loading

  • disable git hooks, .husky folder scripts, by exiting early or commenting out the script
  • is ESLint running on CI? maybe danger included
  • lint all commits on CI
    • check here for help
  • improve docs on how to lint staged files (npx lint-staged)
  • improve docs on how to enable hooks but ignore them in git (git update-index --skip-worktree and git update-index --no-skip-worktree)
  • improve docs on how to handle a commitlint CI fail with a link to rebase docs and commit script explanation
  • add DangerJS rule to error when .husky changes exists in PR, showcasing that enforcing git hooks has been banned
thlmenezes added a commit to thlmenezes/boladao-t3 that referenced this issue Oct 15, 2022
@thlmenezes
🔧 husky: short circuit all hooks
cfd2740 
replace possible performance slowdown on mandatory hook usage
for CI checks, be free to customize your local setup
TMTecnologia/turborepo-tmtecnologia#1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@thlmenezes