-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Vapor 5 #3229
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
base: main
Are you sure you want to change the base?
Conversation
0xTim
commented
Sep 3, 2024

|
Awesome logo~ |
|
Write up here: https://blog.vapor.codes/posts/the-future-of-vapor/ |
| } | ||
| } | ||
| } | ||
| //import Foundation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe add #warning("TODO") to not forget removing the file.
| } | ||
| } | ||
| } | ||
| //import NIOConcurrencyHelpers |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe add #warning("TODO") to not forget removing the file.
… trait (#3356) * Move websockets behind a trait * Don't pull in TLS stuff unless behind a trait * Move client behind a trait * Add CI step to test traits disabled * Fix typo * Use better name for websocket trait * Fix Sendable issue * D'oh * Update test.yml * Make it compile * Fix crash when running tests with package traits * Fix traits workflows --------- Co-authored-by: Gwynne Raskind <gwynne@vapor.codes>
* Fix websocket trait * Fix Android
* Add multipart trait * Move multipart code to behind a trait
Pull in Crypto 4
Make 's async to allow things like file reading when setting up controllers
* Add `.all` routes builder method for all HTTP methods * Add head, options and a test * Update Tests/VaporTests/RouteTests.swift Co-authored-by: Tim Condon <0xTim@users.noreply.github.com> * Add DocC --------- Co-authored-by: Tim Condon <0xTim@users.noreply.github.com>