Skip to content

Conversation

@muftiev
Copy link

@muftiev muftiev commented Aug 17, 2016

Fix #42

@MadLittleMods
Copy link
Owner

Thanks for the PR @muftiev 😀

Please add some tests that previously fail and now work

@muftiev
Copy link
Author

muftiev commented Aug 17, 2016

Updated tests with new cases 😄

@MadLittleMods
Copy link
Owner

@muftiev Let's split those off into their own preserve-variables-in-media file

@muftiev
Copy link
Author

muftiev commented Aug 17, 2016

Ok, done)

@muftiev
Copy link
Author

muftiev commented Aug 18, 2016

@MadLittleMods Will you accept it? 😄

@MadLittleMods
Copy link
Owner

@muftiev Most likely, will try to get this in by Monday. Feel free to ping

@muftiev
Copy link
Author

muftiev commented Aug 22, 2016

@MadLittleMods Ping to remind you to deal with this PR

@muftiev
Copy link
Author

muftiev commented Aug 24, 2016

@MadLittleMods Really wish to use your module in my project, but I need this fix 😄

@MadLittleMods MadLittleMods merged commit 1a7baad into MadLittleMods:master Aug 24, 2016
@MadLittleMods
Copy link
Owner

MadLittleMods commented Aug 24, 2016

Sorry for the wait, I brushed across this tab in my browser everyday (on my mind).

@muftiev Thanks again for the contribution, I'll get this onto a release pronto.

@muftiev
Copy link
Author

muftiev commented Aug 24, 2016

Thank you!

@MadLittleMods
Copy link
Owner

@muftiev
Copy link
Author

muftiev commented Aug 24, 2016

Great! 😄

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