Skip to content

Flask support!#8

Merged
posborne merged 2 commits into
mainfrom
posborne/make-flask-work
Sep 25, 2025
Merged

Flask support!#8
posborne merged 2 commits into
mainfrom
posborne/make-flask-work

Conversation

@posborne
Copy link
Copy Markdown
Member

@posborne posborne commented Sep 5, 2025

Here move away from app.py at the top-level in favor of an examples directory. We clone the functionality of the bottle example and port it to flask. Things don't work out of the box, and we define a set of hack shims for now that manage to make it so at least a very basic service can be written in flask!


Feel free to make modifications while I'm away if you want or I'll be happy to address feedback when I return. Long term I don't expect we'll want the shim present, but I think for this time it still is valuable to prove out the pieces that are missing.

Here move away from app.py at the top-level in favor of an examples
directory.  We clone the functionality of the bottle example and
port it to flask.  Things don't work out of the box, and we define
a set of hack shims for now that manage to make it so at least
a very basic service can be written in flask!
@posborne posborne requested a review from erikrose September 5, 2025 20:09
Copy link
Copy Markdown
Member

@erikrose erikrose left a comment

Choose a reason for hiding this comment

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

All looks good. Maybe we'll ultimately take out the one pytest.mark.integration to mirror what we did on #7, but I'd rather get this merged now so I can work atop it. Exciting to have Flask going!

* Remove `integration` marker that breaks test runs: `'integration' not found in `markers` configuration option`
* Remove reference to `app.wasm` make target that no longer exists. `make test` should suffice.
@erikrose
Copy link
Copy Markdown
Member

I'm inclined to merge this so we can work further on the stuff it adds. It's no more broken than main, and my usual rule for merging is "makes it better than it was". This qualifies.

@posborne posborne merged commit 2e615d2 into main Sep 25, 2025
1 of 3 checks passed
@posborne posborne deleted the posborne/make-flask-work branch September 25, 2025 19:35
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.

2 participants