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

migrate to featchers-common-hooks v4 #103

Merged

Conversation

Anton-Nikishkin
Copy link
Contributor

Summary

  • Tell us about the problem your pull request is solving.
    Remove deprecation dependency warnings.
  • Are there any open issues that are related to this?
    Update to latest feathers-hooks-common? #94
  • Is this PR dependent on PRs in other repos?
    No

@eddyystop
Copy link
Collaborator

Thank you.

@eddyystop eddyystop merged commit 862fcb0 into feathersjs-ecosystem:master Jun 16, 2018
@Sicria
Copy link

Sicria commented Jul 6, 2018

Could you please release a new version where this is included, still receiving these warnings on @2.0.0.

@eddyystop
Copy link
Collaborator

I'll have to look at why the tests are failing.

@eddyystop
Copy link
Collaborator

I pushed a commit directly to the master branch bumping feathers-hooks-common to ^4.14.1 . Published as v2.0..1.

This was not done as a PR because the tests now fail and I don't see how to fix them without a rewrite. Basically some of the .then() are still suspended when Mocha gets control from the test. They only continue to run after the Mocha test timesout.

This'll have to wait until the rewrite planned for this repo.

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.

3 participants