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

pass import path instead of callable in 'screenshot_apps' #30

Merged
merged 1 commit into from
Dec 25, 2024

Conversation

azmeuk
Copy link
Contributor

@azmeuk azmeuk commented Dec 24, 2024

fixes #29

@azmeuk azmeuk force-pushed the 29-wsgi-apps-fix branch 2 times, most recently from eac5926 to c44b34f Compare December 24, 2024 14:06
@azmeuk
Copy link
Contributor Author

azmeuk commented Dec 24, 2024

I am not sure why the CI fails, since it succeeds locally 🤔

@tushuhei tushuhei self-requested a review December 24, 2024 21:24
tests/test_wsgi_apps.py Outdated Show resolved Hide resolved
tests/test_wsgi_apps/test_default.png Outdated Show resolved Hide resolved
setup.cfg Outdated Show resolved Hide resolved
@azmeuk azmeuk force-pushed the 29-wsgi-apps-fix branch 5 times, most recently from 722ff90 to a0d8c5e Compare December 25, 2024 22:05
@azmeuk
Copy link
Contributor Author

azmeuk commented Dec 25, 2024

This is all green now. I had to force a minimum HTML style and fix the content-type.

@tushuhei tushuhei self-requested a review December 25, 2024 22:19
@tushuhei tushuhei merged commit a4263e7 into tushuhei:main Dec 25, 2024
17 checks passed
@tushuhei
Copy link
Owner

This is awesome! Thank you for the completion.

@azmeuk azmeuk deleted the 29-wsgi-apps-fix branch December 25, 2024 22:58
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.

WSGI applications warning
2 participants