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

conflito nas aspas duplas #477

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Maykopr
Copy link

@Maykopr Maykopr commented Nov 19, 2022

Fiz alteração por backtick porque a função recursiva não rodava no navegador. Troquei a versão do código pela versão mais recente que está na terceira versão do livro.

Fiz alteração por backtick porque a função recursiva não rodava no navegador.
@Denisessouza
Copy link

Trabalho sensacional.

@joaostein
Copy link
Contributor

Olá, @Maykopr! O livro em sua segunda versão utiliza ES5 (leia essa seção para mais detalhes). Por isso, ao invés de utilizar template literals, o correto é apenas substituir os caracteres incorretos e por aspas duplas ", deixando o resto do código exatamente como foi escrito em seu texto original.

Para que eu possa dar merge no seu PR, você poderia, por gentileza, reverter as mudanças e deixar o seu código como demonstrado na imagem abaixo?

image

@Maykopr
Copy link
Author

Maykopr commented Nov 20, 2022

Olá @joaostein, Muito obrigado pelas informações. Fiz as alterações como indicadas e abri outro pull request, fiz isto por falta de familiaridade com o Github e ainda não conheço bem o fluxo de trabalho de um pull request.

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