-
Notifications
You must be signed in to change notification settings - Fork 61
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
Package and tests are broken since release of Werkzeug 1.0.0 #29
Comments
Same problem in from werkzeug import secure_filename, FileStorage |
@maxcountryman @rduplain |
It is not only cumbersome, it seems no longer possible: |
As a workaround you can use something like Attention |
I had to downgrade to |
Pull requests are welcome. |
I thought is was the only one seeing this.: cannot import name 'secure_filename' from 'werkzeug' (/opt/hostedtoolcache/Python/3.7.6/x64/lib/python3.7/site-packages/werkzeug/init.py) #98 |
@maxcountryman #28 was the PR - which you already merged. But it would be very good to have a new release out there. While you technically can pip install from a GitHub URL, it is a major pain to have a GitHub URL in a |
Why is using a link a major pain? I’ve done this for many projects and it’s never been much of a hassle. |
At least I could not find a way to run the tests of my applications successfully on Py36-Py38 with a GitHub URL in my setup.py (on Travis only - locally it worked out):
And I tried for a bit :-) What is the issue with cutting a release? Should not take more than a couple of minutes and would make the life of your users much easier. I finally dropped Py3.6 - and so I am personally fine. So I am not asking for myself. Is there anything I could do to support you? |
I also upgraded to flask 1.1.1 with Werkzeug 1.0 and got the deprecate imports. |
I see your point, but honestly it's just inconvenient. Also, even if a url replacement is a quick fix, that's still manual intervention that folks would have to figure out when their builds break. I am very much in favor of a new release to fix the issue and would also be willing to help with any testing or logistics for that 😄 🚀 |
What exactly is holding up a new release? Latest 0.2.1 still has legacy werkzeug issue. This seems like a trivial fix vs forcing folks to patch an otherwise excellent library. |
Why not using the newer |
can someone fix this issue already? |
This issue was reported since Feb 6, unfortunately, 2 months after, no fix has been applied. My app is broken. Please what's holding the maintainers back from fixing this issue? Or are they no longer making use of this extension? |
You guys are so F**king lazy! just a simple command: |
First, a pull request was sent and he merged the PR which is why your pip install from a git repo would work and not break, @maxcountryman is maintaining the library, you should not go about saying people are lazy if they want the merged PR to be updated with a new release. And being an asshole while letting people know how to pip install from a GitHub repo doesn't make you superman or know it all, I wonder how you collaborate with teams, and @maxcountryman should have closed this issue a long time, you won't come here hiding behind your keyboard and calling people F@@ing Lazy. Next time learn how to be civil about what's not yours and enlighten people properly. |
Cutting a new release is a few clicks, no? |
This isn’t the right forum for this. Everyone, please be civil—also I’d ask that you all remember that open source work is not necessarily paid work: myself and the other maintainers are donating our time without direct monetary compensation. Regardless of how much effort it is or isn’t it’s incredibly rude to demand anyone to do anything on your behalf. |
Werkzeug has moved some long deprecated functions/Classes to separate modules.
Since the release of Werkzeug 1.0.0 this package here is broken, also the tests do not work any more.
I created #28 to fix the broken imports.
Could you please cut a new release after merging the PR?
Thanks a lot!
P.S.: If you activate Travis for this repository, I'd volunteer to create a config file and do some minor housekeeping.
The text was updated successfully, but these errors were encountered: