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

Feature: Add CONTRIBUYENDO document #13

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

jcstr
Copy link
Member

@jcstr jcstr commented Nov 10, 2020

  1. Why is this change neccesary?
    This change adds the document that will serve as a reference for the new
    contributors.

  2. How does it address the issue?
    By adding CONTRIBUYENDO.md document to the repository.

  3. What side effects does this change have?
    None.

1. Why is this change neccesary?
This change adds the document that will serve as a reference for the new
contributors.

2. How does it address the issue?
By adding CONTRIBUYENDO.md document to the repository.

3. What side effects does this change have?
None.
@jcstr jcstr added documentation Improvements or additions to documentation enhancement New feature or request labels Nov 10, 2020
@jcstr jcstr added this to the Versión inicial milestone Nov 10, 2020
@jcstr jcstr self-assigned this Nov 10, 2020
@jcstr jcstr linked an issue Nov 10, 2020 that may be closed by this pull request
Copy link
Member

@denisse-dev denisse-dev left a comment

Choose a reason for hiding this comment

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

images - 2020-11-10T173053 440

CONTRIBUYENDO.md Outdated
Nos gustaría que usaras el template de [Andrea Gómez](https://github.com/da-edra/dotfiles/blob/master/.gitmessage) para el mensaje del commit.

# Gracias por leer este documento
Ya que haz llegado al final de este documento nos gustaría agradecerte por tomarte el tiempo de leerlo y con gusto estamos dispuestas y dispuestos a ayudarte.

Choose a reason for hiding this comment

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

s/haz/has/ no?

Copy link
Member Author

@jcstr jcstr Nov 11, 2020

Choose a reason for hiding this comment

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

Listo, corregido 😄

@SantiagoTorres
Copy link

Hmm el deploy esta fallando, pero este PR no toca nada que tenga que ver con el deploy...

@jcstr
Copy link
Member Author

jcstr commented Nov 11, 2020

@SantiagoTorres La razón por la cual falla es que no encuentra el tema en la carpeta themes, ademas de que se descartó el tema por que ya no se le ha estado dando mantenimiento desde hace poco más de un año. Se trabajará en otro tema más nuevo 😄

$ hugo
7:43:36 PM: Error: Unable to find theme Directory: /opt/build/repo/themes/arch

@denisse-dev
Copy link
Member

Sí, disculpen que el deploy esté fallando, me encuentro trabajando en ello. :)

@denisse-dev
Copy link
Member

Creo que estaría mejor poner el template de commit aquí y no vincularlo a mi perfil, también creo que quizá podríamos simplemente pedir que el título tenga como prefijo el tipo de cambio que realiza el commit ergo feature/ o fix/, qué opinan?

@jcstr
Copy link
Member Author

jcstr commented Nov 12, 2020

@da-edra estoy de acuerdo con lo que mencionas, me dejé llevar por el template que compartiste :D

Ahora mismo me encargo de agregar el template del mensaje del commit aquí y corrijo la sección de los prefijos.

@denisse-dev
Copy link
Member

Muchas gracias @jcstr ! :D

Yo sugeriría simplemente mencionar que sería preferible que su commit contenga como prefijo feature/ fix/, etc. Aunque amo mi template de commits porque es bastante útil creo que podría ser tedioso para las personas que quieran contribuir tener que apegarse a ella, de igual manera creo que podríamos tener mejor una plantilla de Pull Requests (#16 ) que podría ser de bastante utilidad cuando la gente quiera contribuir al proyecto. Qué opinan?

@jcstr
Copy link
Member Author

jcstr commented Nov 24, 2020

Listo @da-edra agregue los cambios necesarios, ya esta actualizado el PR. Sólo queda agregar los cambios y listo :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Crear documentación para contribuidores
3 participants