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
replace missing round function in d3 to fix MapBox #6444
Conversation
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.
Please fix the build issues and write some unit tests for that new function.
You can run |
Codecov Report
@@ Coverage Diff @@
## master #6444 +/- ##
==========================================
- Coverage 73.37% 73.36% -0.01%
==========================================
Files 67 67
Lines 9587 9584 -3
==========================================
- Hits 7034 7031 -3
Misses 2553 2553
Continue to review full report at Codecov.
|
thanks for guiding me with my first PR of more to come |
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.
requested changes has been applied
* replace missing round function in d3 * added unit test * fixed linting (cherry picked from commit 71a713d)
* replace missing round function in d3 * added unit test * fixed linting
This PR to fix issue #6279 as round function in d3 seems to be deprecated.