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

Patch 1 #8

Closed
wants to merge 12 commits into from
Closed

Conversation

RIDOPOCO
Copy link

@RIDOPOCO RIDOPOCO commented Mar 6, 2018

No description provided.

@simobasso
Copy link
Member

ciao @RIDOPOCO,

Grazie per aver trovato il tempo di contribuire :)

Come certamente saprai l'argomento è stato già abbondantemente discusso sia sul chan di telegram che su una issue qua (ref: #6 ) quindi se hai qualche argomentazione in più che non è già stata affrontata puoi continuare la discussione su quella issue.

In generale per le prossime pr ti chiederei gentilmente di:

  1. far partire il branch da master e non da un branch vecchio
  2. fare un rebase su master
  3. dare un messaggio un po' più verboso nel commit (puoi trovare una piccola guida qua)
  4. estendere la pr con il perché vuoi applicare queste modifiche

Grazie

@patrick91 patrick91 closed this Mar 6, 2018
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.

5 participants