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

Healthcheck options #108

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Healthcheck options #108

wants to merge 10 commits into from

Conversation

ryansmithevans
Copy link

Added healthcheck options and tests.
Updated Readme

@codecov-io
Copy link

codecov-io commented Mar 22, 2017

Codecov Report

Merging #108 into master will increase coverage by 0.81%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #108      +/-   ##
============================================
+ Coverage     58.08%   58.89%   +0.81%     
- Complexity       79       85       +6     
============================================
  Files             9        9              
  Lines           303      309       +6     
  Branches         24       24              
============================================
+ Hits            176      182       +6     
  Misses          116      116              
  Partials         11       11
Impacted Files Coverage Δ Complexity Δ
...mode/gradle/plugins/docker/image/Dockerfile.groovy 70.27% <100%> (+1.25%) 25 <3> (+3) ⬆️
...adle/plugins/docker/LegacyDockerfileMethods.groovy 52.38% <100%> (+7.93%) 10 <3> (+3) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 18b230a...11ce236. Read the comment docs.

@ryansmithevans
Copy link
Author

Nobody going to review this?

@ryansmithevans
Copy link
Author

Updated to make dockerfile syntax a bit more predictable

@bjornmagnusson bjornmagnusson self-requested a review April 16, 2017 19:33
@bjornmagnusson bjornmagnusson self-assigned this Apr 16, 2017
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.

3 participants