-
Notifications
You must be signed in to change notification settings - Fork 50
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
runtime: implement building image in Docker runtime #760
base: main
Are you sure you want to change the base?
Conversation
6406644
to
2324588
Compare
2324588
to
4b6e794
Compare
@@ -47,7 +47,7 @@ project: | |||
runtime: | |||
# Optional Docker configuration to run code blocks in a container. | |||
docker: | |||
enabled: false | |||
enabled: true |
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.
TODO: Remove before merge.
@@ -130,11 +130,6 @@ func (sl *Cache[T]) Newest() (T, bool) { | |||
return element.Value.(*listEntry[T]).entry, true | |||
} | |||
|
|||
// Deprecated: Use Newest instead. |
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.
TODO: Move it to a separate PR.
|
This PR adds an ability to build a Docker image in Docker runtime. With
runtime.docker.enabled = true
, it should be possible to execute commands from Markdown files.There are still limitations, for example, it's not possible to share env between commands.