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

Fix Werkzeug dependency for testing with Anaconda Python distribution #40

Conversation

aj-stein-nist
Copy link
Contributor

Closes #38.

In order to fix scitt-community#38, we need to hard
pin the version of Werkzeug used by Flask to avoid the changes after
to how functions are organized in the library's modules.

See PR Sanster/IOPaint#390 with a related solution (not specific to
Anaconda approach, but informative).

See [StackOverflow](https://stackoverflow.com/a/77217971) post for
details.
@aj-stein-nist
Copy link
Contributor Author

@pdxjohnny I tested this a few times against your branch and then rebased against main. I am pointing this at main so you can update/rebase the target branch in #39 and it should work. It won't work for other PR (the stack trace is reporting an unrelated problem from what I can tell), but I can review and assist on that as needed. Let me know.

@pdxjohnny
Copy link
Contributor

Thank you @aj-stein-nist for figuring this out! Turned out to be that run-tests.sh installs the package in development mode and the conda tests as they existed (before I updated them in the other PR) did not install the package in development mode before running the tests.

Copy link
Contributor

@pdxjohnny pdxjohnny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@SteveLasker
Copy link
Contributor

Thanks, @aj-stein-nist, @pdxjohnny for figuring out this annoying issue.

@SteveLasker SteveLasker merged commit a2293ff into scitt-community:main Nov 27, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Conda CI failing on main: ImportError: cannot import name 'url_quote' from 'werkzeug.urls'
4 participants