Skip to content
This repository has been archived by the owner on May 6, 2020. It is now read-only.

main: Avoid duplication in makeVersionString() #555

Merged

Conversation

jodh-intel
Copy link
Contributor

Simplified makeVersionString() to avoid duplicating string literals.

Fixes #554.

Signed-off-by: James O. D. Hunt [email protected]

@coveralls
Copy link

coveralls commented Sep 14, 2017

Coverage Status

Coverage increased (+0.005%) to 97.057% when pulling a2713a1 on jodh-intel:simplify-makeVersionString into 876e3fc on clearcontainers:master.

@sameo
Copy link

sameo commented Sep 14, 2017

LGTM

Approved with PullApprove Approved with PullApprove

@coveralls
Copy link

Coverage Status

Coverage increased (+0.005%) to 96.831% when pulling f1c7897 on jodh-intel:simplify-makeVersionString into 91f2b3e on clearcontainers:master.

1 similar comment
@coveralls
Copy link

coveralls commented Sep 14, 2017

Coverage Status

Coverage increased (+0.005%) to 96.831% when pulling f1c7897 on jodh-intel:simplify-makeVersionString into 91f2b3e on clearcontainers:master.

Simplified makeVersionString() to avoid duplicating string literals.

Fixes clearcontainers#554.

Signed-off-by: James O. D. Hunt <[email protected]>
@coveralls
Copy link

coveralls commented Sep 15, 2017

Coverage Status

Coverage increased (+0.005%) to 96.831% when pulling 44fb418 on jodh-intel:simplify-makeVersionString into 0f8ec51 on clearcontainers:master.

@jodh-intel jodh-intel merged commit bc51f37 into clearcontainers:master Sep 15, 2017
mcastelino pushed a commit to mcastelino/runtime that referenced this pull request Dec 6, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants