-
Notifications
You must be signed in to change notification settings - Fork 3
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
feature/193 making the parameter naming consistent. #217
feature/193 making the parameter naming consistent. #217
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## development #217 +/- ##
==============================================
Coverage 88.00% 88.00%
Complexity 518 518
==============================================
Files 74 74
Lines 1750 1750
Branches 217 217
==============================================
Hits 1540 1540
Misses 134 134
Partials 76 76 |
...n.cef.ui/src/de/dlr/sc/virsat/model/extension/cef/ui/templates/DLRCEFStudyCommandHelper.java
Show resolved
Hide resolved
...n.cef.ui/src/de/dlr/sc/virsat/model/extension/cef/ui/templates/DLRCEFStudyCommandHelper.java
Show resolved
Hide resolved
....extension.cefx.test/src/de/dlr/sc/virsat/model/extension/cefx/migrator/Migrator1v2Test.java
Show resolved
Hide resolved
...sat.model.extension.cefx/src/de/dlr/sc/virsat/model/extension/cefx/migrator/Migrator1v2.java
Show resolved
Hide resolved
and code review also improvement.
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.
Looking good from my side! Thanks!
37c3467
to
e38ef89
Compare
Make parameter naming consistent: upper / lower case.