fchispano / recursos

5 stars 6 forks source link

Actualizar CONTRIBUTING.md #20

Open jslirola opened 8 years ago

jslirola commented 8 years ago

Descripción

A la hora de realizar PR no se elimina la lista de requisitos básicos para comprobar que se cumplan algunas pautas. Podemos especificarlo para una mayor claridad y limpieza del repositorio.

Fichero afectado (opcional)

CONTRIBUTING.md

edgarshurtado commented 8 years ago

Yo lo que había pensado es que para empezar pondría la checklist al final de la plantilla de PR.

Otra cosa que he pensado es que ahora lo estamos poniendo como checkboxes, mientras que podríamos simplemente hacerlo como una lista (ol) recordatorio. Al estilo:

Comprueba que has realizado las siguientes acciones antes de mandar la pull-request

  1. He realizado cambios en una rama independiente de master
  2. He comprobado que no se trata de contenido duplicado
  3. Confirmo que sólo hay 10 tipos de personas en el mundo, las que entienden binario y las que no.
OldGordon commented 8 years ago

Si el check-list mejor al final , además "Descripción " lo pondría al principio arriba.

jslirola commented 8 years ago

Se puede probar y hacer todos los cambios en el mismo commit, solo habría que modificar también el fichero PULL_REQUEST_TEMPLATE.md