Informações pessoais (Ricardo Gallego) e #4

[ricardobg]

Adicionei informações pessoais;
Issue #4 feita (adicionar cores de fundo diferentes).

[ghost]

Comentários

Alguns comentários gerais sobre esse pull request:

1. Parabéns pela atenção aos detalhes! O primeiro commit, por exemplo (961967b), introduziu um erro de alinhamento no código (veja no link do commit), que foi corrigido no segundo commit (e8a184a). Existem alguns outros exemplos que foram observados.

2. As mensagens de commit também ficaram muito boas: estão curtas e completas. A única observação sobre elas é que, se você olhar os outros commits do projeto, irá ver que as mensagens (assim como o restante do código) está em Inglês. É sempre legal seguir a convenção do projeto.

3. Os três commits ficaram com autor unkown (veja em https://github.com/infosimples/meet-the-class/pull/32/commits) por causa de uma configuração que não mencionamos na aula, mas faremos aqui.

4. O ideal, para melhor organização do que acontece no repositório, é que exista um pull request para cada issue. No entanto, este pull request está resolvendo duas issues ao mesmo tempo: a [Everybody] Add personal information #1 e a Use different colors for people depending on their course. #4.

5. Existem algumas pequenas melhorias que poderiam ser feitas sobre a implementação da Issue Use different colors for people depending on their course. #4 -- normal, sempre existem. Veja os comentários aqui: https://github.com/infosimples/meet-the-class/pull/32/files

6. A mensagem do último commit (df8fa10) não está condizente com o que o código faz: a mensagem fala em "alteração das cores de fundo", mas o commit, na realidade, apenas remove uma vírgula que havia sido inserida por engano.

---

Alterações pendentes

Dados os comentários acima, as nossas sugestões de alteração estão abaixo. Faça as alterações nessa ordem; caso contrário, podem acontecer coisas inesperadas.

1. Executar os comandos abaixo, para configurar o Git com seu usuário do GitHub:

git config --global user.name "Ricardo Gallego"
git config --global user.email "email-usado-no-cadastro-do-GitHub"

2. Alterar o arquivo js/main.js de acordo com o comentário feito em https://github.com/infosimples/meet-the-class/pull/32/files

3. Fazer um novo commit.

4. Finalmente, fazer um push e verificar se as alterações foram enviadas para o GitHub.

Observação 1: Para facilitar, nós não vamos separar os commits em dois pull requests, mas isso poderia ser feito com o Git.

Observação 2: Se você tiver alguma dúvida sobre como fazer commits, pushs, etc, dê uma olhada nos slides, ou fale conosco.

[ghost]

Esse else if deveria estar alinhado com o if da linha 163.