Skip to content
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

Document inputModeDidChange() lifecycle hook #553

Merged
merged 3 commits into from
May 25, 2017
Merged

Document inputModeDidChange() lifecycle hook #553

merged 3 commits into from
May 25, 2017

Conversation

YoranBrondsema
Copy link
Collaborator

This hook was not documented in the README and I believe it's important enough to have it there.

@YoranBrondsema
Copy link
Collaborator Author

Not sure what happened that made the build fail? The log for the build on Travis is empty for me :/

@YoranBrondsema
Copy link
Collaborator Author

OK, seems to an issue with wd which is a dependency of saucie. The version in yarn.lock (1.1.3) should be OK (see johanneswuerbach/saucie#65) so maybe a refresh of the cache on Travis would solve the issue?

@YoranBrondsema
Copy link
Collaborator Author

Issue here: #557

@bantic
Copy link
Collaborator

bantic commented May 25, 2017

@YoranBrondsema Thanks for this...I've kicked the Travis build. The repo rename from bustlelabs -> bustle seems like it might also have confused Travis. Here's the in-progress build: https://travis-ci.org/bustle/mobiledoc-kit/builds/226236041

@YoranBrondsema
Copy link
Collaborator Author

@bantic Cool thanks! There are a few other small PRs that are ready for review :-)

@bantic
Copy link
Collaborator

bantic commented May 25, 2017

thanks @YoranBrondsema ! Rebuilding travis fixed the tests. :)

@bantic bantic merged commit 88c5bda into bustle:master May 25, 2017
@bantic
Copy link
Collaborator

bantic commented May 25, 2017

released in v0.10.16

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.

2 participants