Skip to content

Conversation

@mrnugget
Copy link
Contributor

No description provided.

@mrnugget mrnugget requested review from eseliger and ryanslade April 15, 2020 07:50
if err != nil {
logger.DockerStepErrored(repoName, i, err, elapsed)
return nil, errors.Wrap(err, "run docker container")
return nil, errors.Wrapf(err, "Running Docker container for image '%s' failed", step.Image)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe use %q?

@mrnugget mrnugget merged commit 0bc433d into master Apr 15, 2020
@mrnugget mrnugget added the action-exec Everything related to the action exec functionality label Apr 17, 2020
scjohns pushed a commit that referenced this pull request Apr 24, 2023
#176)

* Fix -clear-cache flag when cache is cleared and improve error handling

* Use %q instead of single quotes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action-exec Everything related to the action exec functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants