Skip to content

DOC: Add documentation about cpplint #14890

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

Merged
merged 1 commit into from
Dec 16, 2016

Conversation

gfyoung
Copy link
Member

@gfyoung gfyoung commented Dec 15, 2016

  1. Minor spelling and command errors.
  2. Add that we also use AppVeyor for testing.
  3. With the introduction of cpplint, document it and how to use.

Follow-up to #14814.

@gfyoung
Copy link
Member Author

gfyoung commented Dec 15, 2016

@jreback , @jorisvandenbossche : There's a massive backlog on Travis with a good number of old builds (many of which are mine). Do you think you could cancel them before they start?

@jreback jreback added the Docs label Dec 15, 2016
@codecov-io
Copy link

Current coverage is 85.31% (diff: 100%)

Merging #14890 into master will increase coverage by <.01%

@@             master     #14890   diff @@
==========================================
  Files           144        144          
  Lines         51006      51016    +10   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits          43512      43523    +11   
+ Misses         7494       7493     -1   
  Partials          0          0          

Powered by Codecov. Last update 033d345...db09105

@jreback jreback added this to the 0.19.2 milestone Dec 16, 2016
@jreback jreback merged commit 2083f0d into pandas-dev:master Dec 16, 2016
@jreback
Copy link
Contributor

jreback commented Dec 16, 2016

thanks!

@gfyoung gfyoung deleted the c-style-contrib-doc branch December 16, 2016 15:35
ischurov pushed a commit to ischurov/pandas that referenced this pull request Dec 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants