Skip to content

Revert a recent change to Include - #3072

Merged
jbogard merged 2 commits into
LuckyPennySoftware:masterfrom
lbargaoanu:include_order
May 10, 2019
Merged

Revert a recent change to Include#3072
jbogard merged 2 commits into
LuckyPennySoftware:masterfrom
lbargaoanu:include_order

Conversation

@lbargaoanu

Copy link
Copy Markdown
Contributor

Fixes#3069.

…y included, but I guess I trusted the tests too much, it only works in a certain order
@lbargaoanulbargaoanu changed the title Revert a recent changeRevert a recent change to IncludeMay 7, 2019
@jbogardjbogard added the Bug label May 10, 2019
@jbogardjbogard added this to the v.Next milestone May 10, 2019
@jbogard
jbogard merged commit 7ac09c4 into LuckyPennySoftware:masterMay 10, 2019
@lbargaoanu
lbargaoanu deleted the include_order branch May 12, 2019 04:37
@lock

lockBot commented Jun 12, 2019

Copy link
Copy Markdown

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@locklockBot locked as resolved and limited conversation to collaborators Jun 12, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug in using Include to map derived classes

2 participants

@lbargaoanu@jbogard