Skip to content

ARROW-745: [C++] Allow use of system cpplint#476

Closed
cpcloud wants to merge 1 commit intoapache:masterfrom
cpcloud:ARROW-745
Closed

ARROW-745: [C++] Allow use of system cpplint#476
cpcloud wants to merge 1 commit intoapache:masterfrom
cpcloud:ARROW-745

Conversation

@cpcloud
Copy link
Copy Markdown
Contributor

@cpcloud cpcloud commented Apr 1, 2017

You can do pip install cpplint to take advantage of this.

Copy link
Copy Markdown
Member

@wesm wesm left a comment

Choose a reason for hiding this comment

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

+1, sweet, not having cpplint working on Python 3 has been really annoying

@cpcloud
Copy link
Copy Markdown
Contributor Author

cpcloud commented Apr 1, 2017

@wesm It looks like appveyor has stalled. The log is showing this message:

{"message":"Value cannot be null.\r\nParameter name: content"}

@cpcloud
Copy link
Copy Markdown
Contributor Author

cpcloud commented Apr 1, 2017

It's showing up on all queued builds afterwards as well.

@cpcloud
Copy link
Copy Markdown
Contributor Author

cpcloud commented Apr 1, 2017

looks like it's starting :)

@wesm
Copy link
Copy Markdown
Member

wesm commented Apr 1, 2017

sweet, appveyor not cranky anymore. merging

@asfgit asfgit closed this in 651ea92 Apr 1, 2017
@cpcloud cpcloud deleted the ARROW-745 branch April 1, 2017 17:27
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.

2 participants