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

Test for latest matching. #209

Closed
wants to merge 4 commits into from
Closed

Conversation

yavorsky
Copy link
Member

Add test case to compare all latest plugins with all env plugins.
As we know, env preset without options must has the same behavior as a latest.
I think it might be useful while we recommend change latest to env.
Will prevent cases like #192.

@hzoo
Copy link
Member

hzoo commented Mar 14, 2017

Thanks for the PR, although given we are deprecating preset-latest I didn't think we would be updating it anymore?

@codecov-io
Copy link

codecov-io commented Mar 14, 2017

Codecov Report

Merging #209 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #209   +/-   ##
=======================================
  Coverage   92.85%   92.85%           
=======================================
  Files           3        3           
  Lines         196      196           
  Branches       59       59           
=======================================
  Hits          182      182           
  Misses          9        9           
  Partials        5        5

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 ed59056...59bdbd8. Read the comment docs.

@yavorsky
Copy link
Member Author

@hzoo Yes! I consider this PR as a temporal solution for the transition period.

@yavorsky yavorsky closed this Apr 7, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants