-
Notifications
You must be signed in to change notification settings - Fork 0
Update Modal to version 0.73.107 and update API usage in codegen-on-oss #10
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
Conversation
Reviewer's GuideThis pull request updates the Modal dependency to v0.73.107 in File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
27f0eca to
f4656a2
Compare
Description
This PR updates the Modal library usage in the
codegen-on-ossdirectory to use the latest version (0.73.107) and updates the code to follow the latest API changes.Changes
pyproject.tomlfrom 0.73.51 to 0.73.107include_source=Trueparameter to all App and function declarationsconcurrency_limittomax_containersincodegen_modal_deploy.pycodegen_modal_deploy.pyTesting
The changes have been tested locally to ensure they work with the latest Modal API.
Documentation
Updated the README.md file to include information about the Modal version changes and the new API parameters.
💻 View my work • About Codegen
Summary by Sourcery
Update Modal library to version 0.73.107 and adapt codegen-on-oss project to new API
New Features:
include_source=Trueparameter to Modal App and function declarationsEnhancements:
CI:
Documentation:
Chores:
concurrency_limittomax_containersin deployment scripts