Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pull request de la semana 1 #796

Closed
wants to merge 12 commits into from
Closed

Conversation

Surucobe
Copy link

@Surucobe Surucobe commented Nov 4, 2023

Proceso que segui para crear el pull request

  • Buscar una guia para generar un pull request
  • Crear una nueva rama donde haria los cambios a mi repositorio
  • Fusionar la rama creada para los cambios con la rama principal
  • Hacer commit de los cambios realizados
  • Hacer push de la rama principal al repositorio forkeado de en github
  • Ir al repositorio de la academia y hacer click en pull request
  • Ir a la opcion de comparar fork
  • Seleccionar mi repositorio forkeado
  • Agregar descripcion para el pull request

@Surucobe
Copy link
Author

Surucobe commented Nov 4, 2023

Se agregaron los archivos index.html y style.css para cumplir con los requisitos basicos para el pull request

@Surucobe Surucobe closed this Nov 4, 2023
@Surucobe Surucobe reopened this Nov 4, 2023
@MiguelDecode
Copy link

Considero que deberias de eliminar el archivo README sin extensión. No es correcto tener archivos sin extensión.

@Surucobe
Copy link
Author

Surucobe commented Nov 5, 2023

Si, fue un error mandarlo asi, ya lo acomode en mi fork pero no se si debo hacer el pull request otra vez?

@MiguelDecode
Copy link

No necesitas hacer el Pull Request de nuevo al subir los cambios estos se actualizan de manera automática en el pull request. Quedan pendiente algunos puntos de la tarea como añadir una tabla de markdow con al menos cinco comandos de terminal que hayas aprendido y una sección con por ejemplo dos alias que puedas usar o te resulten útiles.

@MiguelDecode
Copy link

Tienes mas archivos de los que se pedía en la práctica y algunos tampoco coinciden en nombre.

Copy link

@MiguelDecode MiguelDecode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Considero que cumple con lo básico.

Copy link
Contributor

🚨🪨 Este pull request está obsoleto porque ha estado abierto durante 10 días sin actividad. Por favor, elimine la etiqueta o el comentario "stale" o se cerrará en un plazo de 5 días

@github-actions github-actions bot added the stale Stale issues or pull requests label Nov 27, 2023
@github-actions github-actions bot closed this Dec 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale Stale issues or pull requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants