duckduckgo-images-api icon indicating copy to clipboard operation
duckduckgo-images-api copied to clipboard

Added max results support

Open stuartspotlight opened this issue 5 years ago • 2 comments

moved over to python3, make the search function return image urls and made the search function limit the number of urls returned

stuartspotlight avatar Jul 11 '18 14:07 stuartspotlight

Did I do this correctly? This is the first time I've attempted to do a pull request

stuartspotlight avatar Jul 11 '18 14:07 stuartspotlight

imo it is better to keep the change to minimal

notes:

  • keep the whitespace for now, even if it doesn't pass flake8
  • keep keywords variable
  • keep printJson function
  • maybe do different method based on the max_results variable. if user don't give any max_inputs value, use the old method, if it is given, use the new one

see other note that i put on code

rachmadaniHaryono avatar Jul 24 '18 13:07 rachmadaniHaryono