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

Reto #30 - Python #4372

Merged
merged 3 commits into from
Jul 31, 2023
Merged

Reto #30 - Python #4372

merged 3 commits into from
Jul 31, 2023

Conversation

Nicolaslaslass
Copy link
Contributor

@Nicolaslaslass Nicolaslaslass commented Jul 30, 2023

Reto #30 - Python

Describe tus cambios

segunda participación en retos semanales

Comprobaciones

Asegúrate de cumplir los siguientes puntos antes de realizar la "Pull Request":

  • El título de mi Pull Request sigue este formato: "Reto #[número] - [lenguaje_utilizado]". (Ej: Reto #0 - Kotlin")
  • El nombre el fichero que se corresponde con el de mi usuario en GitHub más la extensión del lenguaje. (Ej: mouredev.kt)
  • El fichero de corrección se encuentra dentro del directorio del ejercicio y en una carpeta con el nombre del lenguaje de programación utilizado en minúsculas. (Ej: Reto #0/kotlin/mouredev.kt)
  • He revisado que el nombre del directorio del lenguaje no es conflictivo:
    • c#, no csharp
    • c++, no cplusplus
    • go, no golang
    • javascript, no js
  • Únicamente he incluído los ficheros de ejercicios. No se aceptarán Pull Requests que contengan archivos adicionales asociados a editores de código o semejantes.

Información

Copy link
Collaborator

@Roswell468 Roswell468 left a comment

Choose a reason for hiding this comment

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

Hola, por favor revisa y actualiza el nombre del fichero de tu solución (tiene doble extensión .py).
Gracias.
Saludos.

@Nicolaslaslass
Copy link
Contributor Author

Hola, ahí corregí, muchas gracias!

@Roswell468 Roswell468 merged commit 04f601c into mouredev:main Jul 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants