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

Factory WIP #96

Open

brianleroux wants to merge 2 commits into main from factory

Member

brianleroux commented Sep 24, 2021

Common feedback that folks want a scoped factory constructor for begin/data. Usage as follows:

let { Table, factory } = require('@begin/data')

// constructor style
let cats = new Table('cats')
await cats.set({ meow: 'hi' })

// factory style
let [cats, dogs] = factory('cats', 'dogs')
await Promise.all([
  cats.set({ hi: 'meow' }),
  dogs.set({ hi: 'woof' })
])

brianleroux added 2 commits

September 24, 2021 14:19


          first pass adding Table constructor and factory style instantiation

96d0609


          factory polish

15d18e6

Member

tbeseda commented Sep 27, 2021

Coming from other libraries, I like having this option, and the usage looks great 👍

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

Labels

None yet