search

jazzband / geojson

Python bindings and utilities for GeoJSON
https://pypi.python.org/pypi/geojson/
BSD 3-Clause "New" or "Revised" License
905 stars 120 forks source link

Fix PR #109. #112

Closed JanLikar closed 6 years ago

JanLikar commented 6 years ago

Fix tests to make https://github.com/frewsxcv/python-geojson/pull/109 mergeable.

JanLikar commented 6 years ago

Please, keep in mind: I'm not completely familiar with the codebase, so I cannot be confident @jlaxson's code is correct. I merely fixed flake8 issues and a couple of obvious bugs.

codecov-io commented 6 years ago

Codecov Report

Merging #112 into master will decrease coverage by 3.67%. The diff coverage is 45.45%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #112      +/-   ##
=========================================
- Coverage   96.17%   92.5%   -3.68%     
=========================================
  Files          11      11              
  Lines         340     360      +20     
  Branches       59      67       +8     
=========================================
+ Hits          327     333       +6     
- Misses          7      17      +10     
- Partials        6      10       +4
Impacted Files Coverage Δ
geojson/utils.py 81.18% <45.45%> (-12.64%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 0c2b6fe...6ec6aaf. Read the comment docs.

frewsxcv commented 6 years ago

bors r+

bors[bot] commented 6 years ago

Build succeeded