Skip to content

#18-python #3297

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

Merged
merged 1 commit into from
May 1, 2024
Merged

#18-python #3297

merged 1 commit into from
May 1, 2024

Conversation

nandaAlf
Copy link
Contributor

@nandaAlf nandaAlf commented Apr 30, 2024

Describe tus cambios

(Opcional) Sobre todo aconsejable si la "Pull Request" se corresponde con una corrección adicional y no con la presentación de un ejercicio.

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

@kontroldev kontroldev merged commit 0e651c8 into mouredev:main May 1, 2024
@worlion
Copy link
Contributor

worlion commented May 1, 2024

Hola,

Disculpa que me meta, estoy aprendiendo Python y he mirado el código del resto de compis para ayudarme, incluido el tuyo. Creo que hay un error en tu código, en la linea 34, al hacer:

myList[2:3]=['c','d']

Te cepillas el elemento que había en la posición 2 (el '2'). Otro compi hacía lo mismo pero poniendo [2:2] y no le pasa:

myList[2:2]=['c','d']

image

Espero que no te moleste, es por si te sirve.

Un saludo! 🙂

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.

3 participants