Skip to content

Conversation

@Furgas
Copy link
Contributor

@Furgas Furgas commented Jul 24, 2018

Observed in #36.

@clue clue added the bug label Jul 24, 2018
@clue clue added this to the v1.3.1 milestone Jul 24, 2018
@clue
Copy link
Owner

clue commented Jul 24, 2018

@Furgas Thank you for the very quick fix! The changes LGTM, can you add a small test to verify that rejecting with a scalar value (string) throws the expected exception?

@Furgas
Copy link
Contributor Author

Furgas commented Jul 24, 2018

I don't like git, git don't likes me. Well...
I've added checks for UnexpectedValueException message to existing tests.

@clue
Copy link
Owner

clue commented Apr 9, 2019

@Furgas Thank you for the update and your patience, I've just cherry-picked the relevant changes from your PR via #42! :shipit:

@Furgas Furgas deleted the fix-get-class-warning branch April 9, 2019 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants