Conversation
Collaborator
Author
|
@levtcode Ci corre bien pero no esta obligatorio, es importante que lo pongas, estamos en verde tanto en MacOS como en Ubuntu. una cosa, revisa un |
Owner
|
Ya esta hecho, he activado la proteccion de ramas en main. Ahora antes de actualizar los cambios en main, la rama debe estar actualizada a la ultima y tambien se necesitara que todo este en verde antes de realizar el merge |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Acabo de agregar el setup general con todas las cosas para comenzar y con su respectivo dueño.
CMakeListsya no debe tocarse porque ya esta hecho. No tengo permisos para agregar CI para obligar a que todo este verde antes de entrar a main, necesitaría que @levtcode lo hagas manualmente desde la configuración del repo y protección de ramas, agregues una protección para la rama por defectomainen nuestro caso.