Skip to content
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

[MRG+1] Tweak the CSVFeedSpider documentation #2826

Merged
merged 1 commit into from Jul 24, 2017
Merged

Conversation

@dguo
Copy link
Contributor

@dguo dguo commented Jul 14, 2017

I was confused by the headers description, so this is my attempt to make it clearer.

@codecov-io
Copy link

@codecov-io codecov-io commented Jul 14, 2017

Codecov Report

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

@@           Coverage Diff           @@
##           master    #2826   +/-   ##
=======================================
  Coverage   84.73%   84.73%           
=======================================
  Files         163      163           
  Lines        9178     9178           
  Branches     1364     1364           
=======================================
  Hits         7777     7777           
  Misses       1149     1149           
  Partials      252      252
@kmike kmike changed the title Tweak the CSVFeedSpider documentation [MRG+1] Tweak the CSVFeedSpider documentation Jul 14, 2017
@kmike kmike added the docs label Jul 14, 2017
@kmike
Copy link
Member

@kmike kmike commented Jul 14, 2017

It sounds better to me, thanks!

@dangra dangra merged commit 8e0b640 into scrapy:master Jul 24, 2017
2 checks passed
2 checks passed
codecov/patch Coverage not affected when comparing 15abc0b...dedc4a8
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@kmike kmike added this to the v1.5 milestone Dec 22, 2017
@dguo dguo deleted the dguo:patch-1 branch Feb 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.