-
Notifications
You must be signed in to change notification settings - Fork 115
docs: add Runtime Authoring Guide #1890
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
8064137
to
e956dff
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1890 +/- ##
==========================================
- Coverage 47.01% 46.94% -0.08%
==========================================
Files 223 223
Lines 27659 27669 +10
==========================================
- Hits 13005 12989 -16
- Misses 13653 13684 +31
+ Partials 1001 996 -5 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
docs/runtime-implementation-guide.md
Outdated
- If attachStdio=true, enable interactive session wiring where platform supports it (does not impact return semantics). | ||
- Return 0 for stdio transport, or the published host port for SSE/Streamable HTTP. | ||
|
||
List |
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.
Is List part of the workload lifecycle? We have a similar section on listing above in the Runtime contract section, I wonder if we need it in both places.
…ntract Signed-off-by: Juan Antonio Osorio <ozz@stacklok.com>
e956dff
to
1659d3d
Compare
This PR adds a new implementation-agnostic Runtime Authoring Guide.
Key points:
Document path:
This should help future runtime authors implement consistent behavior across platforms while leaving room for platform-specific optimizations.