Codecov Report

Merging #29 into dev will increase coverage by 0.09%. The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev      #29      +/-   ##
==========================================
+ Coverage   66.86%   66.95%   +0.09%     
==========================================
  Files         159      174      +15     
  Lines        4943     5314     +371     
  Branches      456      492      +36     
==========================================
+ Hits         3305     3558     +253     
- Misses       1532     1645     +113     
- Partials      106      111       +5

Impacted Files	Coverage Δ
src/codeschool/questions/text/apps.py	`100% <ø> (ø)`	:arrow_up:
src/codeschool/questions/text/factories.py	`76.47% <ø> (ø)`	:arrow_up:
src/codeschool/core/views.py	`25.16% <ø> (ø)`	:arrow_up:
...hool/questions/coding_io/tests/test_integration.py	`91.36% <ø> (ø)`	:arrow_up:
src/codeschool/conftest.py	`79.54% <ø> (ø)`	:arrow_up:
src/codeschool/lms/academic/api.py	`100% <ø> (ø)`	:arrow_up:
src/codeschool/mixins.py	`55.76% <ø> (ø)`	:arrow_up:
src/codeschool/lms/activities/models/mixins.py	`83.33% <ø> (ø)`	:arrow_up:
src/codeschool/accounts/tests/test_integraton.py	`100% <ø> (ø)`	:arrow_up:
src/codeschool/accounts/viewsets.py	`100% <ø> (ø)`	:arrow_up:
... and 70 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update c831ab4...61ba98a. Read the comment docs.

fabiommendes commented 7 years ago

Pessoal, virou um PR gigantesco (+5000 -50.000!). Me ajudem quebrando em pedaços.

Pelo que entendi aqui tem a contribuição de todo mundo, né? Me mande cada trabalho em um PR separado para que eu consiga administrar a contribuição de cada um.

Imagino que boa parte das alterações são da branch do Ricardo/Fazollino. Eles estavam trabalhando em uma branch instável (ng=nova geração) com a nova versão do Codeschool. Esta branch quebra um monte de coisas e tem um diff gigantesco porque movi todos os arquivos de frontend que estavam espalhados para uma única pasta. Eles precisam fazer o PR nesta branch.

Se ficar difícil separar a contribuição de cada trabalho, separem por app: um PR para o sparta, outro para o blog e outro para a parte de login/usuários.

jonnatas commented 7 years ago

Irei corrigir o pull request e enviar somente o app da Sprint final.

Obter o Outlook para Androidhttps://aka.ms/ghei36

From: Fábio Macêdo Mendes notifications@github.com Sent: Wednesday, July 12, 2017 7:29:12 AM To: cslms/cs-server Cc: jonnatas; Author Subject: Re: [cslms/cs-server] Blog (#29)

Pessoal, virou um PR gigantesco (+5000 -50.000!). Me ajudem quebrando em pedaços.

Pelo que entendi aqui tem a contribuição de todo mundo, né? Me mande cada trabalho em um PR separado para que eu consiga administrar a contribuição de cada um.

Imagino que boa parte das alterações são da branch do Ricardo/Fazollino. Eles estavam trabalhando em uma branch instável (ng=nova geração) com a nova versão do Codeschool. Esta branch quebra um monte de coisas e tem um diff gigantesco porque movi todos os arquivos de frontend que estavam espalhados para uma única pasta. Eles precisam fazer o PR nesta branch.

Se ficar difícil separar a contribuição de cada trabalho, separem por app: um PR para o sparta, outro para o blog e outro para a parte de login/usuários.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/cslms/cs-server/pull/29#issuecomment-314721633, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFE5YWnhAhcoBZN8MYMB95-T_O8-GOLtks5sNJ_4gaJpZM4OVJYl.

cslms / cs-server

Blog #29

Codecov Report