Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(polygon): Improve speed of Polygon.getSignedArea() #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ibgreen
Copy link
Contributor

@ibgreen ibgreen commented Aug 17, 2024

@coveralls
Copy link

Pull Request Test Coverage Report for Build 10433413865

Details

  • 14 of 36 (38.89%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.1%) to 84.862%

Changes Missing Coverage Covered Lines Changed/Added Lines %
modules/polygon/src/polygon-utils.ts 7 29 24.14%
Totals Coverage Status
Change from base Build 10433220835: -0.1%
Covered Lines: 14813
Relevant Lines: 17606

💛 - Coveralls

@smcallis
Copy link

Sweet thanks for trying to get this upstreamed.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants