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

Parse opening hours #57

Open
wants to merge 25 commits into
base: master
Choose a base branch
from

Conversation

KonfrareAlbert
Copy link

First approximation to parse opening hours

@humitos
Copy link
Owner

humitos commented May 24, 2015

Hola,

Primero que nada: "Muchas gracias por colaborar con el proyecto y por usarlo en España!". Luego de dicho eso, te quería comentar algunas cositas sobre "Como colaborar en Github": la idea que es que sigamos estos lineamientos

  • utilizar un pull request por cada nuevo feature (caracteristica) incluída: esto permite que si hay algo que no es compatible con el core del código, se pueden aceptar los pull requests que sí
  • agregar issues al repositorio humitos/osm-pois que indiquen en cuál es el problema o el nuevo feature antes de empezar a trabajar: de esa forma todos vamos a estar informados y podemos colaborar
  • enviar pull requests que apliquen para "todos los sitio": por ejemplo, si hay un feature específico para España (algo en catalán, por ejemplo), para ser enviado debería estar en Español / Castellano para que sea aceptado

No te desanimes, está buenísimo lo que has hecho. Simplemente nos tenemos que organizar un poco para que increíble trabajo nos sirva a todos. Hiciste features muy buenos! Dale para adelante!

Saludos!!

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.

2 participants