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

Correções e melhorias no capítulo como_ler #4

Merged
merged 4 commits into from
Jul 20, 2020
Merged

Conversation

SpawW
Copy link
Contributor

@SpawW SpawW commented Jul 20, 2020

No description provided.

@SpawW
Copy link
Contributor Author

SpawW commented Jul 20, 2020

Adicionei também um parágrafo associando os PRs pequenos e um hormônio da felicidade: a dopamina.


Uma dica aqui é fazer com que seu PR seja o menor possível, pois quanto menor a possibilidade de retrabalho no retorno da revisão é menor.

Outro ponto positivo de um PR mínimo está associado a dopamina que é um dos "hormônios da felicidade", como você já deve ter percebido os jogos que mais jogamos costumam ter fases curtas mas investimos horas e mais horas com eles em nossas telas, por qual motivo? Dopamina, pequenas recompensas neuronais que nos dizem que conseguimos alcançar um objetivo. Então quando estiver fazendo um PR, ao invés de tentar fazer uma fase longa e complexa, tente fazer "fases menores", mais atômicas, e corrija coisas pequenas e rápidas (e da forma correta é claro!) para que você e o avaliador dos PRs possam fazer bom uso de pequenas e constantes doses de dopamina ao aprovar ou ser aprovado cada PR.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tem como coloca uma virgula ali naquele mas?

"jogamos costumam ter fases curtas mas"

@SpawW
Copy link
Contributor Author

SpawW commented Jul 20, 2020

Acho q ajustei corretamente, confere aí por favor.

@gomex gomex merged commit f2daec1 into gomex:main Jul 20, 2020
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