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

Ensure that lock is actually acquired in plugin runner #44

Merged
merged 2 commits into from Apr 8, 2019

Conversation

Projects
None yet
3 participants
@varunvarma
Copy link
Contributor

commented Mar 28, 2019

Check that the lock is actually locked, even if the lock object exists

@varunvarma varunvarma changed the title Fixes yahoo/panoptes#43 Ensure that lock is actually acquired in plugin runner Mar 28, 2019

@coveralls

This comment has been minimized.

Copy link

commented Mar 28, 2019

Pull Request Test Coverage Report for Build 163

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 75.953%

Totals Coverage Status
Change from base Build 161: 0.0%
Covered Lines: 3367
Relevant Lines: 4433

💛 - Coveralls

@rexfury-of-oath rexfury-of-oath merged commit b03099e into yahoo:master Apr 8, 2019

3 checks passed

cla/licenses User has a valid Oath CLA
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 75.953%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.