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

20.09.2023 - Entrega de Exercício (Conta Bancária) #41

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

Conversation

t4lu
Copy link

@t4lu t4lu commented Sep 21, 2023

Critérios de aceitação:

  • Fiz o fork do repositório;
  • Clonei o fork na minha máquina (git clone url-do-meu-fork);
  • Criei minha branch (git checkout -b nome-sobrenome);
  • Criei a pasta com o meu nome dentro da pasta entregas ( mkdir nome-sobrenome);
  • Resolvi o exercício dentro da minha pasta. Como no exemplo;
  • Adicionei as mudanças. (git add . para adicionar todos os arquivos, ou git add nome_do_arquivo para adicionar um arquivo específico);
  • Commitei a cada mudança significativa ou na finalização do exercício (git commit -m "Mensagem do commit");
  • Pushei os commits na minha branch (git push origin nome-da-branch);
  • Criei um Pull Request seguindo as orientações que estão nesse documento.

Extra

  • Validação dos testes executados localmente com o método console.log():

image

  • Validação dos testes executados com o framework Jest:

image

@@ -0,0 +1,65 @@
class ContaBancaria {

Choose a reason for hiding this comment

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

Que código maravilhoso. Ele está muito profissional, eu amei que você utilizou até as nomenclaturas da moeda para deixar o mais real possível. s2

Copy link
Author

Choose a reason for hiding this comment

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

Muito obrigada, Roxanie! ♥

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

2 participants