thadeusb / flask-cache

Cache extension for Flask
http://packages.python.org/Flask-Cache/
Other
700 stars 185 forks source link

fix the #177 #182

Closed ponytailer closed 2 years ago

ponytailer commented 6 years ago

fix the #177 : when the function has the magic arguments, the make_memorize_cache_key has the bug.

ponytailer commented 6 years ago

The CI has been down.

dengshaochun commented 6 years ago

174

error msg

$ python --version
Python 2.7.13
$ pip --version
pip 9.0.1 from /home/travis/virtualenv/python2.7.13/lib/python2.7/site-packages (python 2.7)
0.28s$ pip install . --use-mirrors
Usage:   
  pip install [options] <requirement specifier> [package-index-options] ...
  pip install [options] -r <requirements file> [package-index-options] ...
  pip install [options] [-e] <vcs project url> ...
  pip install [options] [-e] <local project path> ...
  pip install [options] <archive url/path> ...
no such option: --use-mirrors
The command "pip install . --use-mirrors" failed and exited with 2 during .
ponytailer commented 6 years ago

Got it, thanks

ponytailer commented 6 years ago

But, maybe the author has not accepted the new PR.

ponytailer commented 6 years ago

Now, memorize support the magic arguments, so remove and fixed those test case.