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

#31 - Python #5335

Merged
merged 3 commits into from
Aug 6, 2024
Merged

#31 - Python #5335

merged 3 commits into from
Aug 6, 2024

Conversation

avcenal
Copy link
Contributor

@avcenal avcenal commented Aug 5, 2024

Describe tus cambios

Corrección en la clase MainFunctions() para utilizar la instancia de InformsGenerator() correctamente Corrección de varias funciones retirando el return, no era necesario ya que los parámetros pasan por referencia.

Comprobaciones

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

  • El título de mi Pull Request sigue este formato: "#[número] - [lenguaje_utilizado]". (Ej: #00 - Python")
  • El nombre el fichero que se corresponde con el de mi usuario en GitHub más la extensión del lenguaje. (Ej: mouredev.py)
  • 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: 00/python/mouredev.py)
  • 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 incluido los ficheros de ejercicios. No se aceptarán Pull Requests que contengan archivos adicionales asociados a editores de código o semejantes.

Información

Corrección en la clase MainFunctions() para utilizar la instancia de InformsGenerator() correctamente
Corrección de varias funciones retirando el return, no era necesario ya que los parámetros pasan por referencia.
Correcciones mínimas
@kontroldev kontroldev merged commit 45770c7 into mouredev:main Aug 6, 2024
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