Skip to content

Conversation

MartinCarpentier
Copy link
Contributor

@MartinCarpentier MartinCarpentier commented Dec 21, 2021

Updated the readme to reflect the readme from GeoJSON.Net, but changed to reflect the new .Text contents.

Furthermore also included benchmarks comparing GeoJSON.NET and GeoJSON.Text, with actual benchmarks only visible if clicking the collapsible.

Example of readme can be seen here

Readme

@MartinCarpentier MartinCarpentier force-pushed the feature/update-readme branch 3 times, most recently from 0b0541a to 628c6bb Compare December 21, 2021 14:18
@codecov-commenter
Copy link

codecov-commenter commented Dec 21, 2021

Codecov Report

Merging #5 (a3aceca) into main (d0bd715) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main       #5   +/-   ##
=======================================
  Coverage   58.96%   58.96%           
=======================================
  Files          31       31           
  Lines         753      753           
  Branches      192      192           
=======================================
  Hits          444      444           
  Misses        222      222           
  Partials       87       87           
Flag Coverage Δ
unittests 58.96% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


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 d0bd715...a3aceca. Read the comment docs.

@MartinCarpentier MartinCarpentier force-pushed the feature/update-readme branch 5 times, most recently from 73ea979 to ad27030 Compare December 21, 2021 15:46
Copy link
Member

@matt-lethargic matt-lethargic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Top notch

@MartinCarpentier MartinCarpentier merged commit 57d015a into main Dec 23, 2021
@MartinCarpentier MartinCarpentier deleted the feature/update-readme branch December 27, 2021 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants