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

Fix find() issue #26

Merged
merged 1 commit into from Nov 9, 2015

Conversation

Projects
None yet
6 participants
@FranckKe
Contributor

FranckKe commented Nov 2, 2015

Got the fix from #24 and its fork https://github.com/jacobrs/save-commands
All credits goes to @jacobrs. Let me know if you want to do your own PR, I figured you were busy and did it myself.
Fix #25

@jacobrs

This comment has been minimized.

jacobrs commented Nov 2, 2015

@FranckKe I have been quite busy lately and completely forgot to do this, thanks for doing it!

JsonHunt added a commit that referenced this pull request Nov 9, 2015

@JsonHunt JsonHunt merged commit e53f0a2 into JsonHunt:master Nov 9, 2015

@tv-brendan-jefferis

This comment has been minimized.

tv-brendan-jefferis commented Nov 22, 2015

This change has not been included in the latest version of the official package (installed via Atom).

In the meantime, I've manually edited the code myself but it'd be great to see this go out!

@PaulCapestany

This comment has been minimized.

PaulCapestany commented Dec 31, 2015

+1

@chikamichi

This comment has been minimized.

chikamichi commented Jan 7, 2016

It could be nice releasing this fix 🐵

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