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

chore(husky): adding husky git hooks #451

Merged
merged 1 commit into from
Jun 10, 2022
Merged

Conversation

andrefd17
Copy link
Contributor

No description provided.

@vercel
Copy link

vercel bot commented Jun 10, 2022

@andrefd17 is attempting to deploy a commit to the TabNews Team on Vercel.

To accomplish this, @andrefd17 needs to request access to the Team.

Afterwards, an owner of the Team is required to accept their membership request.

If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

@andrefd17
Copy link
Contributor Author

@filipedeschamps,

CommitLint rodando local:

image

Como dar bypass nos hooks:

image

git commit -m "yolo!" --no-verify
HUSKY=0 git push # yolo!

@filipedeschamps
Copy link
Owner

Sensacionaaaaal!!!! 😍 😍 😍

Duas coisas:

  1. O package-lock.json você criou com qual versão do Node.js? Você poderia criar um com a 16.15.0? As dependências estão se comportando de forma doida, não sabemos o que está acontecendo, mas com esta versão no CI está estável: [Error] Arvore de dependencias com incompatibilidade de versões entre primer e versão utilizada do react #445
  2. Você pode dar um squash em todos os commits e mandar um push -f?

@andrefd17
Copy link
Contributor Author

Sensacionaaaaal!!!! 😍 😍 😍

Duas coisas:

  1. O package-lock.json você criou com qual versão do Node.js? Você poderia criar um com a 16.15.0? As dependências estão se comportando de forma doida, não sabemos o que está acontecendo, mas com esta versão no CI está estável: [Error] Arvore de dependencias com incompatibilidade de versões entre primer e versão utilizada do react #445
  2. Você pode dar um squash em todos os commits e mandar um push -f?

Acredito que foi com a 16.14, vou fazer o squash e verificar a versão do node 👍

@filipedeschamps
Copy link
Owner

Merged! Isso resolve o que foi combinado na issue #435 depois do problema que tivemos (e temos ainda) no histórico da main.

/cc @aprendendofelipe

Let's gooooo!!

@filipedeschamps
Copy link
Owner

Não pera, eu fechei o PR!!??!?

@filipedeschamps filipedeschamps restored the husky-git-hooks branch June 10, 2022 17:31
@filipedeschamps filipedeschamps merged commit f9a3383 into main Jun 10, 2022
@filipedeschamps filipedeschamps deleted the husky-git-hooks branch June 10, 2022 17:34
@filipedeschamps
Copy link
Owner

Agora sim oficialmente merged!! Desculpa o vacilo 😂 sexta-feira ao vivo! 🤝

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.

None yet

2 participants