search

argob / cuidar-android

App Cuidar
https://www.argentina.gob.ar/aplicaciones/coronavirus
GNU General Public License v3.0
74 stars 21 forks source link

Se cambió el nombre de TextUtils a ApiConstants para una mejor comprensión, se agregó como compilar la app con firebase #27

Closed gastsail closed 4 years ago

gastsail commented 4 years ago

Se hicieron dos cambios

gastsail commented 4 years ago

Algún update @cuidarapp para este PR ?

cuidarapp commented 4 years ago

Una versión de este PR se incorporó en el branch pre-3.5.4-11-08-2020 y ya fue mergeada en el repo oficial. Gracias por el aporte!

joaquini commented 4 years ago

Sí, realmente el manejo que hacen de los PRs es lo menos ortodoxo que vi en mi vida. Creo que hacen squash de todos los commits internos con cada nueva subida de código. Realmente es lamentable que no puedan mantener un repo Open Source de la manera en que cualquier proyecto Open Source es mantenido: Hasta el equipo de desarrollo activo debería subir PRs. Pero no, see hace esta cosa de diferenciación entre Commit Oficial y PR de la comunidad. El primero pasa sin ningún tipo de filtro. El segundo es cerrado, copiado y mergeado. Un desastre.

Personalmente no creo volver a aportar más nada a este papelón de repositorio.

On Wed, Aug 12, 2020 at 9:38 AM Gastón Saillén notifications@github.com wrote:

no entiendo, pero no mergearon el aporte que hice, no tiene sentido tirar PRs si van a copiar y pegar el código sin mergearlo. No tiene sentido contribuir...

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/argob/cuidar-android/pull/27#issuecomment-672845355, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA7F4MXD3MOZ66XUMF2T6I3SAKELBANCNFSM4PST3JTQ .

gastsail commented 4 years ago

Sí, realmente el manejo que hacen de los PRs es lo menos ortodoxo que vi en mi vida. Creo que hacen squash de todos los commits internos con cada nueva subida de código. Realmente es lamentable que no puedan mantener un repo Open Source de la manera en que cualquier proyecto Open Source es mantenido: Hasta el equipo de desarrollo activo debería subir PRs. Pero no, see hace esta cosa de diferenciación entre Commit Oficial y PR de la comunidad. El primero pasa sin ningún tipo de filtro. El segundo es cerrado, copiado y mergeado. Un desastre. Personalmente no creo volver a aportar más nada a este papelón de repositorio. … On Wed, Aug 12, 2020 at 9:38 AM Gastón Saillén @.***> wrote: no entiendo, pero no mergearon el aporte que hice, no tiene sentido tirar PRs si van a copiar y pegar el código sin mergearlo. No tiene sentido contribuir... — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#27 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA7F4MXD3MOZ66XUMF2T6I3SAKELBANCNFSM4PST3JTQ .

Si, realmente no dan ganas de aportar, estuve esperando 2 semanas para que mergeen esto y poder seguir subiendo contribuciones y hacen un copy paste de lo que subi en vez de directamente mergearlo a un nuevo branch

Lo correcto huebiera sido rechazar el PR y pedir reabrirlo en el nuevo branch @cuidarapp