7.4 KiB

Raw Blame History

Cómo abrir una Pull Request (PR)

Una pull request (PR) le permite enviar cambios desde su bifurcación en GitHub al repositorio principal de freeCodeCamp.org. Una vez que haya hecho cambios en el código, puede seguir estas pautas para abrir un PR.

[!NOTE] Tu PR debe ser en inglés. Consulta aquí para saber cómo contribuir con las traducciones.

Prepara un buen título para tu PR

Recomendamos usar titulos y mensajes convencionales para tus commits y pull request. La convención tiene el siguiente formato:

<tipo>([ámbito opcional]): <descripción>

Por ejemplo:

fix(learn): tests for the do...while loop challenge

Whenever you open a Pull Request(PR), you can use the below to determine the type, scope (optional), and description.

Tipo:

Tipo	Cuándo seleccionar
fix	Funcionalidades actualizadas o mejoradas, pruebas, redacción de la lección, etc.
feat	Sólo si está añadiendo nuevas funcionalidades, pruebas, etc.
chore	Cambios que no están relacionados con el código, las pruebas o la redacción de la lección.
docs	Cambios al directorio `/docs` o a las pautas de contribución, etc.

Ámbito:

Puede seleccionar un ámbito de esta lista de etiquetas.

Descripción:

Keep it short (less than 30 characters) and simple; you can add more information in the PR description box and comments.

Algunos ejemplos de buenos títulos de PR serían:

fix(a11y): contraste mejorado en la barra de búsqueda
feat: se agregan más test a los retos de HTML y CSS
fix(api,client): evitar errores CORS en el envío de formularios
docs(i18n): fix links to be relative instead of absolute

Proponer una Pull Request

Una vez que las ediciones hayan sido confirmadas, se le pedirá que cree un pull request en la página de GitHub de su fork.
De manera predeterminada, todas las pull requests deben estar en contra de la rama freeCodeCamp main repo, main.

Asegúrese de que su bifurcación base está configurada en freeCodeCamp/freeCodeCamp al elevar una solicitud de Pull Request.
Envía el pull request de tu rama a la rama main de freeCodeCamp.
Include a more detailed summary of the changes you made and how your changes are helpful in the body of your PR.
- Se le presentará una plantilla de pull request. Esta es una lista de verificación que debería haber seguido antes de abrir la solicitud de pull request.
- Rellene los detalles como considere oportuno. Esta información será revisada y los revisores decidirán si su pull request es o no aceptada.
- Si el PR está pensado para abordar un problema existente de GitHub entonces, al final de el cuerpo de la descripción de su PR, use la palabra clave Cierra con el número de incidencia para cerrar automáticamente ese problema si se acepta y fusionan las relaciones públicas.
  
  Ejemplo: Cerrar #123 cerrará el problema 123
Indica si has probado en una copia local del sitio o no.
- Esto es muy importante cuando se hagan cambios que no sean solo ediciones del contenido de texto como documentación o una descripción de un desafío. Ejemplos de los cambios que necesitan pruebas en el entorno local incluyen JavaScript, CSS o HTML que podrían cambiar la funcionalidad o el diseño de una página.
- Si tu PR afecta el comportamiento de una página, debe ir acompañada de la correspondiente pruebas de integración de Cypress.

Comentarios sobre pull requests

🎉 Enhorabuena por hacer una PR y muchas gracias por tomarse el tiempo para contribuir.

Nuestros moderadores ahora echarán un vistazo y te dejarán tus comentarios. Por favor, sea paciente con los demás moderadores y respete su tiempo. Todos los pull requests son revisados a su debido tiempo.

And as always, feel free to ask questions on the 'Contributors' category on our forum or the contributors chat room.

[!TIP] Si vas a contribuir con más PRs, te recomendamos que leas las directrices "haciendo cambios y sincronizando" para evitar la necesidad de borrar tu bifurcación.

Conflicto en un pull request

Los conflictos pueden surgir porque muchos colaboradores trabajan en el repositorio y los cambios pueden romper su PR que está pendiente de revisión y fusión.

La mayoría de las veces podrías no requerir un rebase porque juntamos todas las confirmaciones, sin embargo, si un rebase es solicitado, esto es lo que tienes que hacer.

Para arreglos de errores y características habituales

Cuando trabajas con errores regulares y funcionalidades en nuestra rama de desarrollo main, puedes hacer un simple rebase:

Reiniciar su copia local:

git checkout <pr-branch>
git pull --rebase upstream main

Resolver cualquier conflicto y añadir / editar commits

# O bien
git add .
git commit -m "chore: resolve conflicts"

# Or
git add .
git commit --amend --no-edit

Enviar los cambios al PR
```
git push --force origin <pr-branch>
```

Para los próximos planes de estudios y características

Cuando estás trabajando en características para nuestras próximas ramas del plan de estudio next-*, tienes que hacer un "cherry pick":

Asegúrese de que su autor está sincronizado con su local:

git checkout main
git fetch --all --prune
git checkout next-python-projects
git reset --hard upstream/next-python-projects

Respaldar

a. Borra tu rama local después de realizar una copia de seguridad (si todavía la tienes localmente):

git checkout <pr-branch-name>

# ejemplo:
# git checkout feat/add-numpy-video-question

git checkout -b <backup-branch-name>

# ejemplo:
#  git checkout -b backup-feat/add-numpy-video-question

git branch -D <pr-branch-name>

b. O simplemente una copia de seguridad de su rama pr (si no la tiene localmente):

git checkout -b <backup-branch-name> origin/<pr-branch-name>

# ejemplo:
#  git checkout -b backup-feat/add-numpy-video-question origin/feat/add-numpy-video-question

Empezar con una pizarra limpia:

git checkout -b <pr-branch-name> next-python-projects
git cherry-pick <commit-hash>

Resuelve cualquier conflicto y limpia, instala pruebas de ejecución

npm run clean

npm ci
npm run test:curriculum --superblock=<superblock-name>

# ejemplo:

# npm run test:curriculum --superblock=python-for-everyone

Si todo se ve bien empuje hacia el PR

git push --force origin <pr-branch-name>

7.4 KiB Raw Blame History