-
Notifications
You must be signed in to change notification settings - Fork 640
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
Conversation
…los requisitos basicos para el pull request
Se agregaron los archivos index.html y style.css para cumplir con los requisitos basicos para el pull request |
Considero que deberias de eliminar el archivo README sin extensión. No es correcto tener archivos sin extensión. |
Si, fue un error mandarlo asi, ya lo acomode en mi fork pero no se si debo hacer el pull request otra vez? |
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. |
…tml para cumplir con la tarea
… se creo una carpeta para guardar la practica de la clase de css
Tienes mas archivos de los que se pedía en la práctica y algunos tampoco coinciden en nombre. |
There was a problem hiding this 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.
🚨🪨 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 |
Proceso que segui para crear el pull request