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

Added typescript to the project! #58

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

AssisrMatheus
Copy link

I took the time to add Typecript to the project!

Unfortunately, we can't debug and type the search term at the same time, since the readline-sync crashes(something about TTY). So this method is helpful only for IDE static typing.

@AssisrMatheus AssisrMatheus changed the title 📦 Added typescript to the project! Added typescript to the project! Mar 18, 2019
@maycrodrigues
Copy link

maycrodrigues commented Mar 18, 2019

Opa! Tenho também o mesmo projeto em TS, só não fiz o PR ainda. É que na verdade não sei se vale faz o PR e mudar tanto o projeto assim...

https://github.com/maycrodrigues/video-maker-typescript

@AssisrMatheus
Copy link
Author

Opa! Tenho também o mesmo projeto em TS, só não fiz o PR ainda. É que na verdade não sei se vale faz o PR e mudar tanto o projeto assim...

https://github.com/maycrodrigues/video-maker-typescript

Fala maycro! Manero!

Então, eu tentei mudar o projeto o menos possível. Vi no seu projeto que você criou classes separadas de user input e inclusive para os robots.

No meu caso, eu somente adicionei os tipos nas funções, e criei interfaces para a estrutura de dados do projeto(content e sentence). Diminuindo o footprint do typescript no projeto e mantendo ele o máximo próximo do original possível, nem passei os arquivos para a pasta "src", como você fez.

Parece que você criou o seu do zero, muito legal! Eu só adicionei o typescript no código existente mesmo, não criei do zero não. Legal que a gente consegue ver as várias formas diferentes de fazer o mesmo resultado.

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