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

Jump to bottom

Unexpected keyword argument in new_process_output(...) #4576

Closed
yash-nisar opened this issue Jul 28, 2017 · 1 comment · Fixed by #4579
Closed

Unexpected keyword argument in new_process_output(...) #4576

yash-nisar opened this issue Jul 28, 2017 · 1 comment · Fixed by #4579
Assignees
@yash-nisar
Labels
area/CI area/tests type/bug

Comments

@yash-nisar
Copy link
Member

By @jayvdb:
new_process_output() got an unexpected keyword argument 'cc_threshold'

https://travis-ci.org/coala/docker-coala-base/builds/256537062

This may have been a hidden error, exposed by recent improvement to coala core testing, or maybe something more sinister.
@yash-nisar
Copy link
Member Author

This occurs because kwargs passed to process_output(...) have not been taken care of.

@yash-nisar yash-nisar mentioned this issue Jul 28, 2017
Closed
yash-nisar added a commit to yash-nisar/coala that referenced this issue Jul 29, 2017
@yash-nisar
LocalBearTestHelper: Add **process_output_kwargs
2868324 
Add `**process_output_kwargs` argument to `new_process_output(...)`
and `old_process_output(...)` because the keyword arguments were
not taken care of in bbb99f9.

Fixes coala#4576
Related to coala/docker-coala-base#219
@yash-nisar yash-nisar mentioned this issue Jul 29, 2017
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yash-nisar yash-nisar

Labels
area/CI area/tests type/bug
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

LocalBearTestHelper: Add **process_output_kwargs yash-nisar/coala
3 participants
@Makman2 @yash-nisar @gitmate-bot