search

xgcm / xhistogram

Fast, flexible, label-aware histograms for numpy and xarray
https://xhistogram.readthedocs.io
MIT License
90 stars 20 forks source link

Add some more info to contributing.rst #37

Closed dougiesquire closed 3 years ago

dougiesquire commented 3 years ago

Added a few more details to contributing.rst. Mostly things that I've found useful when trying to get started. Much of this is taken from xskillscore documentation so thanks to those who helped to develop that.

codecov[bot] commented 3 years ago

Codecov Report

Merging #37 (904cab0) into master (bb1fa23) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #37   +/-   ##
=======================================
  Coverage   94.41%   94.41%           
=======================================
  Files           3        3           
  Lines         215      215           
  Branches       58       58           
=======================================
  Hits          203      203           
  Misses          8        8           
  Partials        4        4

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 bb1fa23...904cab0. Read the comment docs.

dougiesquire commented 3 years ago

Thanks for the review @jrbourbeau! @rabernat are you okay to merge?