search

mattlisiv / newsapi-python

A Python Client for News API
MIT License
376 stars 130 forks source link

added exclude domain parameter #24

Closed jborchma closed 5 years ago

jborchma commented 5 years ago

Hey Matt,

when going through the API docs I noticed that the excludeDomains keyword was missing from the Python API get_everything method. I added it in the same form as the other keywords had been added and included testing for the new variable in the same form as you had done for the others. I also added an entry into the docstring.

I just noticed my text editor also got rid of whitespaces in empty lines and at the end of the lines, I hope that's ok too. So the main changes are around lines 180.

Let me know what you think!

Cheers, Jan

mattlisiv commented 5 years ago

Hi @jborchma,

Thanks for the addition! Merged.

Best, Matt