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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update events to the latest version 馃殌 #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

greenkeeper[bot]
Copy link

@greenkeeper greenkeeper bot commented May 25, 2018

Version 3.0.0 of events was just published.

Dependency events
Current Version 2.1.0
Type dependency

The version 3.0.0 is not covered by your current version range.

If you don鈥檛 accept this pull request, your project will work just like it did before. However, you might be missing out on a bunch of new features, fixes and/or performance improvements from the dependency update.

It might be worth looking into these changes and trying to get this project onto the latest version of events.

If you have a solid test suite and good coverage, a passing build is a strong indicator that you can take advantage of these changes directly by merging the proposed change into your project. If the build fails or you don鈥檛 have such unconditional trust in your tests, this branch is a great starting point for you to work on the update.


Release Notes v3.0.0

This version drops support for IE8. events no longer includes polyfills
for ES5 features. If you need to support older environments, use an ES5 shim
like es5-shim. Both the shim and sham
versions of es5-shim are necessary.

  • Update to events code from Node.js 10.x
    • (semver major) Adds off() method
  • Port more tests from Node.js
  • Switch browser tests to airtap, making things more reliable
Commits

The new version differs by 18 commits.

  • d0e217c 3.0.0
  • 5c06c27 Merge pull request #52 from Gozala/node-10
  • ebd9686 Disable iPhone test for now, SauceLabs is not running it for some reason
  • f1750fd Add MS Edge to test matrix.
  • 8b763d2 Update airtap to 0.0.6
  • ae8c267 Print MaxListenersExceeded warnings as TAP comments.
  • c3e91f9 Add events-list and special-event-names test; more robust Reflect polyfills
  • 6e1abfa Update readme: clarify Node version, shims, feature contribution policy
  • 38d5a78 Polyfill Number.isNaN.
  • 59d3074 Skip method-name tests if Function.name is not available
  • 567f4d3 Add airtap.local loopback and set sauce_connect: true.
  • c40bd27 Update readme.
  • 4121c64 Switch to airtap
  • e3df7ce Use tape
  • a795d60 Run tests in latest node.

There are 18 commits in total.

See the full diff

FAQ and help

There is a collection of frequently asked questions. If those don鈥檛 help, you can always ask the humans behind Greenkeeper.


Your Greenkeeper bot 馃尨

greenkeeper bot added a commit that referenced this pull request Jan 8, 2020
@greenkeeper
Copy link
Author

greenkeeper bot commented Jan 8, 2020

  • The dependency events was updated from 2.1.0 to 3.1.0.

Update to this version instead 馃殌

Release Notes for v3.1.0

events now matches the Node.js 11.12.0 API.

  • pass through return value in wrapped emitter.once() listeners

    Now, this works:

    emitter.once('myevent', function () { return 1; });
    var listener = emitter.rawListeners('myevent')[0]
    assert(listener() === 1);

    Previously, listener() would return undefined regardless of the implementation.

    Ported from nodejs/node@acc506c

  • Reduce code duplication in listener type check (#67 by @friederbluemle).

  • Improve emitter.once() performance in some engines

Commits

The new version differs by 15 commits.

  • 4f03fbf 3.1.0
  • 0754cd5 improve emitter.once() performance
  • 7ad7ba6 pass through return value in onceWrapper()-ed functions
  • ade7481 remove dollar symbol for private function
  • 3b825ea Consolidate listener checks (#67)
  • 0a84716 Consolidate listener checks
  • 7b91104 Add FUNDING.yml.
  • 203bd09 Add security.md
  • 21a6ea5 Disable package-lock.json.
  • a7eb04c Merge pull request #59 from Gozala/airtap-1
  • 9b3853b Update airtap to v1.0.0.
  • bda6f8a Merge pull request #57 from Gozala/airtap-0.0.9
  • d598613 Update airtap to 0.0.9.
  • f6f6a37 Merge pull request #55 from Gozala/fix/iphone
  • cd39248 Update airtap and add iPhone back to test matrix

See the full diff

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.

None yet

0 participants