GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Proceso de Revisión · Issue #286 · javascript-tutorial/es.javascript.info

Proceso de Revisión #286

New issue
Copy link
New issue
Copy link
Open
Open
Proceso de Revisión#286
Copy link

Description

@EzequielCaste
EzequielCaste
opened on Jun 29, 2020
Issue body actions

Para coordinar las review que estamos realizando, creo que necesitamos un procedimiento bien delineado:

  1. Solicitar un Reviewer
    cualquiera de los miembros: @vplentinax @joaquinelio @EzequielCaste @lizzie136 @ricardov03
    Si eres miembro y vas a hacer la revisión, por favor asignarse el PR antes de comenzar.

  2. El reviewer revisa y corrige errores si existen.
    Con el botón "Add your review"
    Tienen la opción de Comentar, Aprobar o Solicitar Cambios.

  3. Solicitar Cambios (Request Changes)
    Recomiendo utilizar la sugerencia de cambios para hacer la tarea más eficiente. Es fácil y rápido sugerir un cambio y el autor con un click puede incluir esa sugerencia al PR.

  4. Aprobación
    Una vez que el traductor realiza los cambios solicitados, en reviewer aprueba la traducción. El bot le colocará el tag de "needs +1" que nos ayudará a ver los PR que están aprobados y listo para merge.

Los tags nos deberían servir para ver el estado de los PR:
needs review >> changes requested >> needs +1

Reacted by Valentiina VP

Activity

  1. joaquinelio commented on Jun 29, 2020

    @joaquinelio
    joaquinelio
    on Jun 29, 2020
    Member
    More actions

    ¿Asignarse? ¡Oh!, Aprendí algo nuevo, minimizaría la superposición, PERFECTO, dar control a Don BOT y requerir menos coordinación humana, gracias.
    ¿Qué hay con reviewers invitados?
    Hoy por hoy necesitamos más revisión que traducción, es frustrante aportar traducción y que no te respondan en meses.
    Si sabe hescrivir vien y detectar errores de concepto de js, ¡100% de aumento! De trabajo.

  2. EzequielCaste commented on Jun 29, 2020

    @EzequielCaste
    EzequielCaste
    on Jun 29, 2020
    ContributorAuthor
    More actions

    image

    Click en el engranaje y se puede agregar reviewer: Request up to 15 reviewers

  3. joaquinelio commented on Jun 29, 2020

    @joaquinelio
    joaquinelio
    on Jun 29, 2020
    Member
    More actions

    @ezzep66
    Veo 2 cosas y no sé la diferencia,
    reviewers
    assignees
    o para que sirve "assigne" , crei que era lo mismo y te referias a ese.

    en un pr aparezco como reviewer no se si alguien me solicitó o cuando terminé git lo puso solo,
    y me pude agregar como assignee "assign yourelf" pero ni idea de para qu´e
    el doc no ayuda nada

  4. joaquinelio commented on Jun 29, 2020

    @joaquinelio
    joaquinelio
    on Jun 29, 2020
    Member
    More actions

    help.github.com/en/github/managing-your-work-on-github/assigning-issues-and-pull-requests-to-other-github-users
    no sirve
    https://developer.github.com/v3/issues/assignees/
    no sirve

    google "whats the point in github assignee"
    stackoverflow:
    actually meant to be used with issues rather than pull request so that when we receive a issue we can assign someone to fix it
    OK!!!! Ahora entiendo que efectivamente te referías a reviewer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      You are visitor 01402520 — GeezerVison does not store the pages you look up.
      Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.