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

Agrega comportamiento hoisting #22

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Agrega comportamiento hoisting #22

wants to merge 1 commit into from

Conversation

josoroma-zz
Copy link

Empieza neuronas alrededor del comportamiento hoisting.

Preguntas, comentarios, sugerencias => bienvenidas 👍

connect to #6

@josoroma-zz
Copy link
Author

Un por typo ahi: alguien no lleva tilde :)

@gaboesquivel
Copy link
Member

para hacer correcciones en pull request, simplemente subes tus cambios al branch del desde el cual estas haciendo el PR. Es importante que hagas squash de tus commits utilizando el git rebase...
Squash significa comprimir varios commits en uno solo. En mi opinion la forma más sencilla es corriéndolo con el flag -i... Ejemplo: quieres comprimir los últimos 2 commits en uno solo corres git rebase -i HEAD~2 ... ver http://davidwalsh.name/squash-commits-git

@josoroma-zz
Copy link
Author

Pura vida Gabo, sip ya lo he abusado bastante el squash, me gusta mas asi de hecho, por ahi he estado mejorando muchas cosas en los 5 pull requests, en unos estaba diciendo muchas caballadas :)

Ustedes harian el "squash" con git rebase -i master (interactivo parece mas facil) y seguidamente un push del branch de trabajo asi git push -f?

Cuando se coopera la primera vez en un tema no hace falta, pero cuando el pull request no ha sido aprobado aun y uno hace mejoras hay que forzarlo, conocen otra manera mas decente de lograrlo o asi esta bien?

Tuanis!

@josoroma-zz
Copy link
Author

Ya me auto-conteste con el "RTFM" de David Walsh que habias compartido (y)

@gaboesquivel
Copy link
Member

@josoroma me refería a sqashing los commits en tu branch (josoroma:feature/conceptos-fundamentales/hoisting), para que el pull-request venga limpio.. únicamente con commit relevantes, 1 por cambio lógico.

@josoroma-zz
Copy link
Author

@gaboesquivel
Copy link
Member

solo veo un commit tuyo,
ya hiciste squash de los correcciones ortográficas y typos que mencionabas?

@josoroma-zz
Copy link
Author

Sip todo mejorado y esquasheado!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants