DFOP PHTG - Atividade 5 #253
Conversation
|
Pedro e Dyego, este pull contém vários arquivos que vão contra a observação "Pull requests não devem conter arquivos gerados ou de configuração particular de cada máquina ou IDE, como .classpath, .project, .class. O mesmo vale para arquivos de log e similares. Não deve-se dar commits nesses arquivos." de https://sites.google.com/a/cin.ufpe.br/desenvolvimento-de-software-taes-2013-1/instrues-para-colaborao-com-o-git-para-o-rgms. Favor corrigir. |
| @@ -44,7 +44,7 @@ class NewsController { | |||
|
|
|||
| def save = { | |||
| if (!grailsApplication.config.grails.mail.username) { | |||
| throw new RuntimeException(message(code: 'mail.plugin.not.configured', 'default': 'Mail plugin not configured')) | |||
| throw new RuntimeException(message(code: 'mail.plugin.not.configured', 'default': 'Mail plugin not configured') as Throwable) | |||
There was a problem hiding this comment.
Por que foi necessário o as?
There was a problem hiding this comment.
Foi uma sugestão do próprio Intellij quando demos um Inspect Code
|
Alguns desses arquivos apareceram depois que os testes foram rodados. Não deveriam estar no pull, não. Foi uma falha nossa. Vou corrigir e reenviar o mais rápido possível. |
|
Professor, Corrigi o pull request, retirando os arquivos indesejados. Fechei o pull request antigo e abri um novo seguindo o mesmo padrão. Fiz isso apenas para garantir que o pull request estaria correto. Aguardo um segundo feedback para modificar o que for preciso. |
Pull Request dos alunos:
Dyego Felipe de Oliveira Penha
Pedro Henrique Torres Gonçalves
Cenários de GUI e de controlador criados para a feature de 'news'.
Issue #252 criada e corrigida. Um novo cenário foi criado para cobrir ela.