-
Notifications
You must be signed in to change notification settings - Fork 0
Adding new feature to geo utils #321
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
Conversation
This comment has been minimized.
This comment has been minimized.
Codecov ReportBase: 67.31% // Head: 67.64% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #321 +/- ##
============================================
+ Coverage 67.31% 67.64% +0.33%
- Complexity 231 232 +1
============================================
Files 40 40
Lines 1242 1255 +13
Branches 117 117
============================================
+ Hits 836 849 +13
Misses 373 373
Partials 33 33
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
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.
Thanks for tackling this 🙂
# Conflicts: # CHANGELOG.md
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
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.
Looks good! There are some Codacy code style issues mostly related to groovy syntax features. It would be great if you could fix them otherwise I'm happy 🙂 .
This comment has been minimized.
This comment has been minimized.
Analysis Details0 IssuesCoverage and DuplicationsProject ID: edu.ie3:utils |
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.
Looks good, thanks 🙂
Resolves #320