Skip to content

fix: usage with starlette >= 1#5423

Open
grawlinson wants to merge 1 commit intoplatformio:developfrom
grawlinson:port-starlette-1
Open

fix: usage with starlette >= 1#5423
grawlinson wants to merge 1 commit intoplatformio:developfrom
grawlinson:port-starlette-1

Conversation

@grawlinson
Copy link
Copy Markdown

In 1.0.0, starlette has removed the on_startup/shutdown handlers (Kludex/starlette#3117) in favour of a lifespan handler. This commit implements support for starlette >= 1.

However, I have not touched platformio/dependencies.py since I do not know what this project's policy is on dependencies. Let me know if you'd like that implemented also.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 31, 2026

CLA assistant check
All committers have signed the CLA.

Comment thread platformio/home/run.py Outdated
@dekeonus
Copy link
Copy Markdown

I added support for starlette-1.0.0 to gentoo's build for platformio-core.
see:
https://github.com/gentoo/gentoo/blob/28f6b4b79aad17eb8938bd70cd3fa1911dd51469/dev-embedded/platformio/files/pio-6.1.19-starlette-1.patch

starlette has removed the on_startup/shutdown handlers in favour of a
lifespan handler. This commit implements support for starlette >= 1.
@grawlinson
Copy link
Copy Markdown
Author

I added support for starlette-1.0.0 to gentoo's build for platformio-core. see: https://github.com/gentoo/gentoo/blob/28f6b4b79aad17eb8938bd70cd3fa1911dd51469/dev-embedded/platformio/files/pio-6.1.19-starlette-1.patch

I've just replaced the PR with this commit, as it's a lot better ... not to mention that it actually works!

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.

3 participants