devschile / huemul

:sparkles: Hubot para devsChile
http://devschile.cl
70 stars 72 forks source link

Se agregan owners por scripts en CODEOWNERS #599

Closed lgaticaq closed 4 years ago

lgaticaq commented 4 years ago

Usuarios confirmados de participar en el CODEOWNERS

lgaticaq commented 4 years ago

@devschile/org :+1: | :-1: :pray:

gmq commented 4 years ago

Es super buena idea pero me da cosa agregar gente que tal vez no está ni ahí y que para cambiarlo sea necesario un PR.

davidlaym commented 4 years ago

nosotros usamos codeowners en algunos repos grandes en la pega y funciona bastante bien.

Me parece que falta el script de uoct si @lgaticaq

lgaticaq commented 4 years ago

Es super buena idea pero me da cosa agregar gente que tal vez no está ni ahí y que para cambiarlo sea necesario un PR.

A todos los PRs se agregan el team devschile/org y de forma adicional se agrega el owner da cada script. Asi si el owner no responde nunca siempre estará el fallback de algún admin

lgaticaq commented 4 years ago

Incluí los test de los scripts al PR, se me habían olvidado

lgaticaq commented 4 years ago

Es super buena idea pero me da cosa agregar gente que tal vez no está ni ahí y que para cambiarlo sea necesario un PR.

A todos los PRs se agregan el team devschile/org y de forma adicional se agrega el owner da cada script. Asi si el owner no responde nunca siempre estará el fallback de algún admin

@gmq eso :point_up: te tranquiliza?

gmq commented 4 years ago

@lgatica no :vv: Mi problema va mas por el lado de agregar gente sin su consentimiento, sobre todo considerando que les va a llegar una notificación cada vez que alguien edite su script hasta que alguien (o ellos mismos) los saquen del CODEOWNERS

Creo que prefiero que sea voluntario el agregarse al archivo, que alguien agregue un script no significa que se quiera hacer cargo de él por toda la eternidad

lgaticaq commented 4 years ago

Dale voy a consultar a cada uno si acepta o no

clsource commented 4 years ago

onda yo no creo tener scripts de huemul xd. Pero si un dia subo alguno podria ver.

lgaticaq commented 4 years ago

onda yo no creo tener scripts de huemul xd. Pero si un dia subo alguno podria ver.

Este script es tuyo o no?

javiera-vasquez commented 4 years ago

Cual es la idea, que cada uno se haga cargo de los script que ha cerrado o tomemos alguno abanonado tambien?

lgaticaq commented 4 years ago

Cual es la idea, que cada uno se haga cargo de los script que ha cerrado o tomemos alguno abanonado tambien?

En primer lugar que cada autor sea asignado como revisor cuando se cree un nuevo PR que afecte al script creado por este. Ademas hay un par de scripts huérfanos que pueden auto asignarse si así lo desean.

La idea es mejorar la participación de la comunidad en la revisión de PRs

inawrath commented 4 years ago

Apoyo sin dramas 👌

gmq commented 4 years ago

Agrégame a karma pls 🙏

lgaticaq commented 4 years ago

@gmq actualice CODEOWNERS y elimine a los que no se pronunciaron y a los que prefirieron no formar parte. En la descripción todos los que aceptaron