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.

Sign up for GitHub

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

Jump to bottom

Release 3.0 #250

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Release 3.0 #250

wants to merge 6 commits into from

Commits on Nov 25, 2021

  1. Release 3.0 CHANGELOG placeholder 

    Initial CHANGELOG placeholder for release 3.0.
    @hwillson
    hwillson committed Nov 25, 2021
    Configuration menu
    Copy the full SHA
    d370843 View commit details
    Browse the repository at this point in the history

  2. Add a(n optional) generic type map to PubSub. (#245) 

    * Add a(n optional) generic type map to PubSub.

The class PubSub now has a generic type parameter so its methods `publish` and `subscribe` can be **optionally** type-checked by TypeScript.

* Added part for PubSub generic.

* Slight README tweaks to align formatting / adjust grammar a bit

* Changelog update

Co-authored-by: hwillson <[email protected]>
    @mchccn @hwillson
    mchccn and hwillson committed Nov 25, 2021
    Configuration menu
    Copy the full SHA
    8e8d511 View commit details
    Browse the repository at this point in the history

  3. feat: replace iterall with native Symbol.asyncIterator + fix return t… 

    …ypes (#232)

* fix(typings): return AsyncIterableIterator instead of AsyncIterator

BREAKING fixes the type annotation of the abstract class PubSubEngine. According to the TypeScript type-defintion a `PubSubAsyncIterator` instance is actually a `AsyncIterableIterator` instead of an  `AsyncIterator`. The typing of `PubSubAsyncIterator` is way more convenient as it allows iterating over it with the `for await (const foo of iterator) { doSth() }` syntax, which is super handy for filtering or mapping (See https://gist.github.com/n1ru4l/127178705cc0942cad0e45d425e2eb63 for some example operators).

* remove iterall

* rename asyncIterator method to asyncIterableIterator.

* Slight tweaks based on graphql-js 16 changes

* Changelog update

Co-authored-by: hwillson <[email protected]>
    @n1ru4l @hwillson
    n1ru4l and hwillson committed Nov 25, 2021
    Configuration menu
    Copy the full SHA
    e27eb2e View commit details
    Browse the repository at this point in the history

  4. fix: Support readonly arrays as event names (#234) 

    * Failing case: pubsub asyncIterator should accept a readonly string[]

* fix: Allow readonly triggers to be passed to PubSubEngine.asyncIterator

* Changelog update

Co-authored-by: hwillson <[email protected]>
    @rh389 @hwillson
    rh389 and hwillson committed Nov 25, 2021
    Configuration menu
    Copy the full SHA
    28934af View commit details
    Browse the repository at this point in the history

  5. Start highlighting breaking changes in the CHANGELOG

    @hwillson
    hwillson committed Nov 25, 2021
    Configuration menu
    Copy the full SHA
    5199043 View commit details
    Browse the repository at this point in the history

  6. Allow resolver fn to be async (#220) 

    * Allow resolver fn to be async

* Changelog updates

Co-authored-by: hwillson <[email protected]>
    @maclockard @hwillson
    maclockard and hwillson committed Nov 25, 2021
    Configuration menu
    Copy the full SHA
    94a9993 View commit details
    Browse the repository at this point in the history