Skip to content

Auto Detect Version #81

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

Merged
merged 4 commits into from
Jul 14, 2020
Merged

Auto Detect Version #81

merged 4 commits into from
Jul 14, 2020

Conversation

yoavrotems
Copy link
Contributor

Change definitions to be under CIS versions name, removed default version from viper, and add auto detect docker version for choosing the correct cis + unit tests

…rsion from viper, and add auto detect docker version for choosing the correct cis + unit tests
@yoavrotems yoavrotems linked an issue Jul 9, 2020 that may be closed by this pull request
Update test var version to be cis-1.2
@yoavrotems yoavrotems changed the title Change definitions to be under CIS versions names, removed default ve… Auto Detect Version Jul 9, 2020
Add function trimVersion to handle with versions suffix.
The check with version.constraints is not working versions prerelease or metadata like docker version 18.09.03-ce
@codecov
Copy link

codecov bot commented Jul 9, 2020

Codecov Report

Merging #81 into master will increase coverage by 5.36%.
The diff coverage is 71.87%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
+ Coverage   45.83%   51.20%   +5.36%     
==========================================
  Files           3        3              
  Lines          96      125      +29     
==========================================
+ Hits           44       64      +20     
- Misses         49       54       +5     
- Partials        3        7       +4     
Impacted Files Coverage Δ
app.go 49.43% <70.96%> (+9.43%) ⬆️
root.go 58.82% <100.00%> (ø)

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 90ee61d...c1adcf9. Read the comment docs.

@yoavrotems yoavrotems requested a review from itaysk July 13, 2020 08:28
@yoavrotems yoavrotems merged commit 49f5ebc into master Jul 14, 2020
@itaysk itaysk deleted the auto-detect-version branch July 14, 2020 11:14
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.

Default version is always 17.06
2 participants