Skip to content

Conversation

russcam
Copy link
Contributor

@russcam russcam commented Sep 30, 2019

Fixes #4105

Copy link
Contributor

@codebrain codebrain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@russcam
Copy link
Contributor Author

russcam commented Sep 30, 2019

failing tests don't appear to be related to this change.

@russcam russcam merged commit b669afd into master Sep 30, 2019
russcam added a commit that referenced this pull request Sep 30, 2019
russcam added a commit that referenced this pull request Oct 1, 2019
@Mpdreamz Mpdreamz deleted the fix/4105 branch November 6, 2019 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[PropertyName("name", Ignore=true)] ignore not honoured.
2 participants