Skip to content
This repository was archived by the owner on Aug 11, 2022. It is now read-only.

fetch-package-metadata,add-named: Unify package version selection #15205

Closed
wants to merge 1 commit into from

Conversation

iarna
Copy link
Contributor

@iarna iarna commented Dec 13, 2016

Supplants #10662.

Gives us a single source of truth when it comes to selecting which version to use when given a range specifier.

@iarna iarna added the review label Dec 13, 2016
@coveralls
Copy link

Coverage Status

Coverage increased (+0.03%) to 85.769% when pulling d83ee74e17586911d4c2f59ed0c9b5a1218a7b2e on iarna/there-can-be-only-one into 190a658 on release-next.

@othiym23
Copy link
Contributor

Add or repurpose some unit tests for the extracted function and this will be ready to go. Nice work!

@iarna iarna force-pushed the iarna/there-can-be-only-one branch from d83ee74 to b6c8a3f Compare December 13, 2016 01:49
@othiym23
Copy link
Contributor

Tests – and everything else – LGTM. 🐑🚀🔥

@coveralls
Copy link

Coverage Status

Coverage increased (+0.03%) to 85.769% when pulling b6c8a3f on iarna/there-can-be-only-one into 190a658 on release-next.

@iarna iarna force-pushed the release-next branch 7 times, most recently from 20e7365 to ff1f0bb Compare December 16, 2016 01:53
@iarna
Copy link
Contributor Author

iarna commented Dec 16, 2016

This was merged and released in 4.1.0.

@iarna iarna closed this Dec 16, 2016
@iarna iarna deleted the iarna/there-can-be-only-one branch March 7, 2017 22:54
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