-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[14.0][MIG] epa_account_custom: Migration to 14.0 #35
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 14.0 #35 +/- ##
==========================================
+ Coverage 79.41% 84.61% +5.20%
==========================================
Files 5 12 +7
Lines 68 91 +23
Branches 10 13 +3
==========================================
+ Hits 54 77 +23
Misses 8 8
Partials 6 6 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bom teste funcional está okay e questões de código também, notei que no commit do IMP ocorreu a adição do arquivo manifest e readme, ele deveria conter somente a parte do setup/
O restante que está envolvido com ele seria no commit de MIG.
Para ajustar essa questão dos commit, roda o comando git reset HEAD~2
, ele vai te trazer os dados dos 2 ultimos commit seu (IMP e MIG), faz o git add do setup/ para o Commit de IMP e o restante para MIG
b13f59b
to
e340f52
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (REVIEW FUNCTIONAL AND CODE)
No description provided.