-
Notifications
You must be signed in to change notification settings - Fork 639
Add support for build secrets #1010
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
494b8ef to
3406011
Compare
3312023 to
b5d244e
Compare
Signed-off-by: Mattt Zmuda <mattt@replicate.com>
| PythonVersion string `json:"python_version,omitempty" yaml:"python_version"` | ||
| PythonRequirements string `json:"python_requirements,omitempty" yaml:"python_requirements"` | ||
| PythonPackages []string `json:"python_packages,omitempty" yaml:"python_packages"` // Deprecated, but included for backwards compatibility | ||
| Run []RunItem `json:"run,omitempty" yaml:"run"` |
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.
Does this require an update to the run docs given the change in syntax? Do we want to continue to support the old syntax as well?
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.
Good callout. I just added docs with 080e0fa.
This new format is an additive change, so existing uses of run commands will continuing working. I don't have any plans to deprecate the old syntax.
Signed-off-by: Mattt Zmuda <mattt@replicate.com>
* Add support for run commands with mounted secrets Signed-off-by: Mattt Zmuda <mattt@replicate.com> * Document new run item format Signed-off-by: Mattt Zmuda <mattt@replicate.com> --------- Signed-off-by: Mattt Zmuda <mattt@replicate.com>
Resolves #1009