#784 warning when resolving unlisted package #1258

Merged
merged 1 commit into from Nov 27, 2015

Conversation

Projects
None yet
2 participants
@alex-bogomaz
Contributor

alex-bogomaz commented Nov 25, 2015

hi. Please review this pull request. Now warning is displayed when unlisted package is resolved

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Nov 26, 2015

Member

I think we should only warn when the resolver terminates with unlisted packages.

Member

forki commented Nov 26, 2015

I think we should only warn when the resolver terminates with unlisted packages.

forki added a commit that referenced this pull request Nov 27, 2015

Merge pull request #1258 from alex-bogomaz/#784-unlisted-warning
#784 warning when resolving unlisted package

@forki forki merged commit 2c7e7bc into fsprojects:master Nov 27, 2015

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Nov 27, 2015

Member

thx for submitting. I move the code a bit. see 88b8023 - does this work for you?

Member

forki commented Nov 27, 2015

thx for submitting. I move the code a bit. see 88b8023 - does this work for you?

@alex-bogomaz

This comment has been minimized.

Show comment
Hide comment
@alex-bogomaz

alex-bogomaz Nov 30, 2015

Contributor

sure, thanks!

Contributor

alex-bogomaz commented Nov 30, 2015

sure, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment