Skip to content

Add Nota private controller [Merge After PR #69] - #71

Open
paulofelipefeitosa wants to merge 2 commits into
masterfrom
nota-private-controller
Open

Add Nota private controller [Merge After PR #69]#71
paulofelipefeitosa wants to merge 2 commits into
masterfrom
nota-private-controller

Conversation

@paulofelipefeitosa

Copy link
Copy Markdown
Member

No description provided.

@codecov

codecovBot commented Mar 3, 2019

Copy link
Copy Markdown

Codecov Report

Merging #71 into master will decrease coverage by 0.24%.
The diff coverage is 64.51%.

Impacted file tree graph

@@ Coverage Diff @@## master #71 +/- ##
============================================
- Coverage 80.42% 80.17% -0.25% - Complexity 281 289 +8 
============================================
Files 36 39 +3 Lines 664 681 +17 Branches 80 81 +1 ============================================
+ Hits 534 546 +12 - Misses 86 88 +2 - Partials 44 47 +3
Impacted FilesCoverage ΔComplexity Δ
.../main/java/br/com/openenade/api/estado/Estado.java66.66% <ø> (ø)12 <0> (ø)⬇️
...openenade/api/modalidade/ModalidadeController.java50% <ø> (ø)1 <0> (ø)⬇️
...e/api/categoriaadmin/CategoriaAdminController.java50% <ø> (ø)1 <0> (ø)⬇️
...rc/main/java/br/com/openenade/api/curso/Curso.java81.08% <ø> (ø)15 <0> (ø)⬇️
...om/openenade/api/modalidade/ModalidadeService.java0% <0%> (ø)0 <0> (?)
...penenade/api/universidade/UniversidadeService.java90.32% <0%> (+7.96%)14 <0> (ø)⬇️
...enade/api/universidade/UniversidadeController.java66.66% <100%> (ø)6 <1> (ø)⬇️
...in/java/br/com/openenade/api/nota/NotaService.java100% <100%> (+5.71%)11 <5> (+1)⬆️
.../com/openenade/api/nota/PrivateNotaController.java100% <100%> (ø)3 <3> (?)
...va/br/com/openenade/api/modalidade/Modalidade.java81.81% <100%> (+4.04%)4 <3> (+1)⬆️
... and 14 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 cf892f8...dfc0a64. Read the comment docs.

@paulofelipefeitosa

Copy link
Copy Markdown
MemberAuthor

This pull request introduces 1 alert when merging dfc0a64 into cf892f8 - view on LGTM.com

new alerts:

  • 1 for Array index out of bounds

Comment posted by LGTM.com

Sign up for freeto 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.

1 participant

@paulofelipefeitosa