spidermon icon indicating copy to clipboard operation
spidermon copied to clipboard

docs: Documenting built-in mixins

Open rennerocha opened this issue 5 years ago • 2 comments

Continue work started on https://github.com/scrapinghub/spidermon/pull/133

rennerocha avatar Nov 26 '19 18:11 rennerocha

Hey @rennerocha I've looking a bit into #133 and this and was wondering if the issue could be up for grabs?

I think documenting SpiderMonitorMixin should include some docs for spidermon.contrib.stats.counters.DictPercentCounter which is mixed with ResponseInfo from mixin class.

I'd be happy to contribute some docs to this.

mirceachira avatar Dec 21 '19 12:12 mirceachira

Codecov Report

Merging #232 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #232   +/-   ##
=======================================
  Coverage   69.28%   69.28%           
=======================================
  Files          67       67           
  Lines        2745     2745           
  Branches      306      306           
=======================================
  Hits         1902     1902           
  Misses        784      784           
  Partials       59       59

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 0623843...811d86c. Read the comment docs.

codecov[bot] avatar Dec 30 '19 14:12 codecov[bot]