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.

Sign up for GitHub

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

Jump to bottom

Adiciona sintetizador de voz #466

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

Adiciona sintetizador de voz #466

wants to merge 2 commits into from

Conversation

rodfersou
Copy link
Member

@rodfersou rodfersou commented Mar 27, 2018 •
edited

Boas práticas para o PR: Checklist

Foi aberta uma issue relativa a esse PR;

  • Sim
  • Não
  • Não se aplica

Foram adicionados testes unitários;

  • Sim
  • Não
  • Não se aplica

Foram adicionados testes robots;

  • Sim
  • Não
  • Não se aplica

Foram adicionados upgradeSteps;

  • Sim
  • Não
  • Não se aplica

Foi adicionada a modificação no CHANGES.rst do pacote, sempre no topo do arquivo (ou seja, na primeira linha do release ainda não lançado), contendo o seu nome de usuário do github e a referência ao issue que você está tratando (não esqueça de colocar no fim do arquivo a url para a issue, use o padrão do próprio arquivo);

  • Sim
  • Não
  • Não se aplica

PR não contém assuntos diferentes. PR's devem ter um objetivo claro para facilitar o review. Ex: não junte, num mesmo PR, alterações de code-analysis e de implementação. Faça um PR de code-analysis e depois o da implementação em si;

  • Sim
  • Não
  • Não se aplica

Evite muitos commits pequenos de um mesmo assunto no PR, sempre que possível efetue rebase;

  • Sim
  • Não
  • Não se aplica

@rodfersou
Copy link
Member Author

issue: #467

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@rodfersou