Skip to content
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

Gracefully handle nil position bug #80

Merged
merged 1 commit into from
May 13, 2016
Merged

Gracefully handle nil position bug #80

merged 1 commit into from
May 13, 2016

Conversation

mattbrictson
Copy link
Owner

Airbrussh::Rake::Context is not thread-safe, and this can cause problems in some Capistrano deployments. For now, work around the problem by defaulting to "01" for the command number in Airbrussh's output when Context is not able to provide the correct info.

Fixes #78

Airbrussh::Rake::Context is not thread-safe, and this can cause problems in some
Capistrano deployments. For now, work around the problem by defaulting to "01"
for the command number in Airbrussh's output when Context is not able to
provide the correct info.
@coveralls
Copy link

coveralls commented May 13, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling 2527905 on nil-position into 1d32a49 on master.

@mattbrictson mattbrictson merged commit 19666a5 into master May 13, 2016
@mattbrictson mattbrictson deleted the nil-position branch January 2, 2017 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants