GithubHelp home page GithubHelp logo

zoom-is-configurable about help-maps-party HOT 5 OPEN

delineas avatar delineas commented on July 18, 2024
zoom-is-configurable

from help-maps-party.

Comments (5)

skcode7 avatar skcode7 commented on July 18, 2024

Saludos Dani
Me apunto para colaborar, voy a revisar un poco el código y luego crearme una rama a ver como me vá.

from help-maps-party.

skcode7 avatar skcode7 commented on July 18, 2024

Que tal Dani,
Te comento que ya hice el cambio y funciona perfecto en local, ahora tengo una duda con respecto al pull-request
Ayer estuve haciendo algunos cambios y los respectivos commits, hoy luego del cambio del zoom hice un nuevo commit y al generar el PR aparecen todos los objetos modificados.
Estuve investigando un poco y al parecer si se puede eliminar los commits, pero no me queda claro el proceso.
imagen

from help-maps-party.

delineas avatar delineas commented on July 18, 2024

La mejor opción es esta:
https://mattstauffer.com/blog/how-to-merge-only-specific-commits-from-a-pull-request/

Con esto lo que harías sería tener en una rama el cambio específico que quieres enviar al repositorio padre.

Eso sí, necesitas poner el repositorio en el mismo punto en el que estaba en origen, antes de introducir tus modificaciones. Esto es, deberías tener una rama (master, por ejemplo) donde no añadirías modificaciones tuyas salvo que quieras que formen parte el repositorio padre.

Todo esto es para evitar pasar de un repo a otro en el PR los cambios que son específicos de tu proyecto.

De esta aprendemos todos :))

from help-maps-party.

skcode7 avatar skcode7 commented on July 18, 2024

Finalmente lo logre! :)
Me tomó varias horas entender los pasos a seguir, pero al final quedó el PR con los cambios correctos.
Bien dicho, aprendemos todos.

from help-maps-party.

delineas avatar delineas commented on July 18, 2024

Gracias! Y felicidades! Ahora me toca a mi repasar lo que has enviado y practicar una saludable y positiva Code Review ;)

from help-maps-party.

Related Issues (9)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.