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

middleware/chaos: fix version #669

Merged
merged 2 commits into from
May 22, 2017
Merged

middleware/chaos: fix version #669

merged 2 commits into from
May 22, 2017

Conversation

miekg
Copy link
Member

@miekg miekg commented May 22, 2017

Move the version setting into a init function so it is done early. Then
tweak the setup code for chaos a bit to correctly pick this version up.
Add an integration test to pick this up in the toplevel test/ directory.

Fixes #667

miekg added 2 commits May 22, 2017 09:51
Move the version setting into a init function so it is done early. Then
tweak the setup code for chaos a bit to correctly pick this version up.
Add an integration test to pick this up in the toplevel test/ directory.

Fixes #667
@codecov-io
Copy link

Codecov Report

Merging #669 into master will increase coverage by <.01%.
The diff coverage is 75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #669      +/-   ##
==========================================
+ Coverage   41.03%   41.04%   +<.01%     
==========================================
  Files         126      126              
  Lines        6463     6464       +1     
==========================================
+ Hits         2652     2653       +1     
+ Misses       3600     3599       -1     
- Partials      211      212       +1
Impacted Files Coverage Δ
coremain/run.go 39.47% <100%> (+1.75%) ⬆️
middleware/chaos/setup.go 60.71% <50%> (-5.96%) ⬇️

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 7e6f5c7...d82a029. Read the comment docs.

@johnbelamaric johnbelamaric merged commit 024f566 into master May 22, 2017
@miekg miekg deleted the chaos branch May 22, 2017 12:46
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