Skip to content

Fix regressions introduced in #1644 and #1657 #1800

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

Conversation

Arkni
Copy link
Member

@Arkni Arkni commented Jun 11, 2016

//Cc @Maks3w

@Arkni Arkni force-pushed the fix-defaultmessage-and-previousvalue branch from d5e6d3f to eaf96db Compare June 11, 2016 22:23
@Maks3w
Copy link
Contributor

Maks3w commented Jun 12, 2016

Good for me.

Just one more thing. @staabm Please add this deprecations to release notes

@staabm
Copy link
Member

staabm commented Jun 14, 2016

@Arkni can we get a unit test into this PR?

@Arkni
Copy link
Member Author

Arkni commented Jun 14, 2016

Sure!
I'll add them as soon as possible.

@Arkni Arkni force-pushed the fix-defaultmessage-and-previousvalue branch from eaf96db to 9d2f9ae Compare June 14, 2016 20:53
@Arkni Arkni force-pushed the fix-defaultmessage-and-previousvalue branch from 9d2f9ae to 986a47d Compare June 14, 2016 20:55
@Arkni
Copy link
Member Author

Arkni commented Jun 14, 2016

@staabm
Added the tests!

@staabm staabm closed this in 74b38e9 Jul 15, 2016
@Arkni Arkni deleted the fix-defaultmessage-and-previousvalue branch July 15, 2016 14:23
@staabm
Copy link
Member

staabm commented Jul 15, 2016

merged, thx!

@Arkni
Copy link
Member Author

Arkni commented Jul 26, 2016

@staabm

we forgot this:

Just one more thing. @staabm Please add this deprecations to release notes

Can you add it please ?

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

Successfully merging this pull request may close these issues.

3 participants