-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Fix --timeout flag #569
Fix --timeout flag #569
Conversation
internal/artifact/run.go
Outdated
ctx := context.Background() | ||
if c.Timeout > 0 { | ||
var cancel context.CancelFunc | ||
ctx, cancel = context.WithTimeout(ctx, c.Timeout) | ||
defer cancel() | ||
} |
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.
Timeout
has the default value.
Line 155 in 020c4a3
Value: time.Second * 120, |
How about the following way?
ctx := context.Background() | |
if c.Timeout > 0 { | |
var cancel context.CancelFunc | |
ctx, cancel = context.WithTimeout(ctx, c.Timeout) | |
defer cancel() | |
} | |
ctx := context.WithTimeout(context.Background(), c.Timeout) | |
defer cancel() |
internal/client/run.go
Outdated
if c.Timeout > 0 { | ||
var cancel context.CancelFunc | ||
ctx, cancel = context.WithTimeout(ctx, c.Timeout) | ||
defer cancel() | ||
} |
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.
ditto
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.
@knqyf263 done
* Wrapped scan call inside context timeout * timeout has default value
* feat: Add hcl syntax highlighting
Issue