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

Removed V3 -> V2 fallback #2782

Merged
merged 1 commit into from Sep 22, 2017

Conversation

Projects
None yet
3 participants
@eugene-g
Contributor

eugene-g commented Sep 20, 2017

See #2508

@forki forki requested a review from matthid Sep 21, 2017

@matthid

To be completely honest I think we had this before and this line is one of those quick-fixes/hacks. We should try again without this (Cannot remember what was broken)

@forki forki merged commit 290d0b8 into fsprojects:master Sep 22, 2017

1 of 2 checks passed

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

This comment has been minimized.

Show comment
Hide comment
@matthid

matthid Sep 22, 2017

Member

Note to future me: Such lines need a comment/link to the related issue :)

Member

matthid commented Sep 22, 2017

Note to future me: Such lines need a comment/link to the related issue :)

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