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

#48-challenge-counting-letters #49

Merged
merged 9 commits into from
Jul 19, 2024

Conversation

CRBM84
Copy link
Collaborator

@CRBM84 CRBM84 commented Jul 4, 2024

This is a pull request for the counting letters challenge. I thinks I figured out the pull request issue.

@CRBM84 CRBM84 self-assigned this Jul 4, 2024
@itsmePo
Copy link
Collaborator

itsmePo commented Jul 4, 2024

¡Holi!
El código, la branch y demás está bien, pero me salta error con tu .gitignore.
Con un par de consultas rápidas al señor GPT me sale que tienes código en el archivo que está en conflicto con la propia branch.
Habría que solucionar eso antes de continuar!

Te dejo un enlace con los pasos a seguir junto al señor GPT.
https://chatgpt.com/share/86cf2cbb-a6c1-4cf0-8332-c734d19923ef

Copy link
Collaborator

@itsmePo itsmePo left a comment

Choose a reason for hiding this comment

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

Pese a que sale todo correcto y el código funciona, tu archivo .gitignore está en conflicto con la rama.
En uno de los comentarios te he dejado un enlace paso a paso para poder solucionarlo! :D

Copy link
Collaborator

@Rafa0297 Rafa0297 left a comment

Choose a reason for hiding this comment

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

Faltaría agregar la condición en la linea 19 para que funcione bien el console.assert()

staff/carlos-bock/challenge-letter/counting-letters.js Outdated Show resolved Hide resolved
@CRBM84
Copy link
Collaborator Author

CRBM84 commented Jul 5, 2024

¡Holi! El código, la branch y demás está bien, pero me salta error con tu .gitignore. Con un par de consultas rápidas al señor GPT me sale que tienes código en el archivo que está en conflicto con la propia branch. Habría que solucionar eso antes de continuar!

Te dejo un enlace con los pasos a seguir junto al señor GPT. https://chatgpt.com/share/86cf2cbb-a6c1-4cf0-8332-c734d19923ef

@itsmePo gracias! Lo voy a ver :) ¿Que clase de error ha salido? :/

Gracias por el enlace lo trato de solucionar.

@CRBM84
Copy link
Collaborator Author

CRBM84 commented Jul 6, 2024

¡Holi! El código, la branch y demás está bien, pero me salta error con tu .gitignore. Con un par de consultas rápidas al señor GPT me sale que tienes código en el archivo que está en conflicto con la propia branch. Habría que solucionar eso antes de continuar!

Te dejo un enlace con los pasos a seguir junto al señor GPT. https://chatgpt.com/share/86cf2cbb-a6c1-4cf0-8332-c734d19923ef

Se ha resuelto el problema. :)

@CRBM84 CRBM84 requested review from Rafa0297 and itsmePo July 6, 2024 13:37
Copy link
Collaborator

@Rafa0297 Rafa0297 left a comment

Choose a reason for hiding this comment

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

Hay que cambiar el nombre de la carpeta /challenge-letters a /challenges

staff/carlos-bock/challenge-letter/counting-letters.js Outdated Show resolved Hide resolved
@CRBM84 CRBM84 requested a review from rucev July 9, 2024 17:02
@CRBM84 CRBM84 changed the title #48 challenge counting letters #48 challenge-counting-letters Jul 15, 2024
@CRBM84 CRBM84 requested a review from Rafa0297 July 15, 2024 16:55
@CRBM84 CRBM84 changed the title #48 challenge-counting-letters #48-challenge-counting-letters Jul 15, 2024
@venturars
Copy link
Collaborator

Esta PR tiene conflictos de merging

@rucev
Copy link
Collaborator

rucev commented Jul 19, 2024

Elimina el console.log dentro de la función

Copy link
Collaborator Author

@CRBM84 CRBM84 left a comment

Choose a reason for hiding this comment

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

Extra console.log removed.

Copy link
Collaborator

@rucev rucev left a comment

Choose a reason for hiding this comment

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

😄

@rucev rucev merged commit d38df5c into b00tc4mp:develop Jul 19, 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.

None yet

6 participants