-
Notifications
You must be signed in to change notification settings - Fork 0
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
README: update define class content #133
Conversation
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.
@pnwpedro Left some feedback. I'll push some changes for streaming shortly.
On second thought, I'll open a separate PR to handle streaming. Thanks! |
okay np. i was thinking we'd queue up one big readme update and merge it during release. smaller is fine, but we prob want to set the default branch on the repo to the most recent release branch so users see the relevant readme when they come here |
Ah — I'll push my changes here then. Added with |
Co-authored-by: James Rodewig <[email protected]>
Co-authored-by: James Rodewig <[email protected]>
Co-authored-by: James Rodewig <[email protected]>
Co-authored-by: James Rodewig <[email protected]>
Co-authored-by: James Rodewig <[email protected]>
Description
Get the readme ready for a beta release
Motivation and context
Because it has to be right! And good!
How was the change tested?
Screenshots (if appropriate):
Change types
Checklist:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.