search

spgroup / rgms

Research Group Management System
11 stars 77 forks source link

DFOP PHTG - Atividade 7 #262

Closed pedrootorres closed 10 years ago

pedrootorres commented 10 years ago

Pull Request dos alunos: Dyego Felipe de Olveira Penha Pedro Henrique Torres Gonçalves

Todas as análises de código feitas para a feature de News. Classe CommonPages excluída como combinado.

pauloborba commented 10 years ago

pelo que vi, a única mudança relativa a modularidade foi a eliminação de uma superclasse? é isso mesmo?

pedrootorres commented 10 years ago

Sim, professor. A feature news, no começo da disciplina, já estava bem organizada. Durante as entregas nós fomos modificando o código sempre seguindo padrões de modularização. Para essa atividade restaram apenas algumas mudanças. Entre elas estavam imports não utilizados, código morto e a classe CommonPages (passamos a função que ela executava para dentro da classe que a chamava).

pauloborba commented 10 years ago

Pessoal, ainda tem algo errado aqui. O rebase não foi feito corretamente. Quando tento integrar o pull de vocês dá uma série de conflitos. Esses conflitos deveriam ter sido resolvidos no rebase.

pedrootorres commented 10 years ago

Professor, eu criei uma nova branch, dei outro rebase e refiz as alterações que nós tínhamos feito para essa atividade. Estou fechando esse pull request porque já dei um novo. Acredito que dessa vez não tenha problemas. Pelo menos conseguir retirar aquele excesso de commits. Se ainda parecer ter alguma coisa errada, pode avisar.