Michaël Zasso

Results 1686 comments of Michaël Zasso

Related: https://github.com/zakodium/adonis-mongodb/issues/72#issuecomment-876122196 If we implement the public `$attributes` field, it will be possible to do `{...instance.$attributes}`

In the mean time, I will make `ownKeys` throw so that spreading by mistake doesn't do something unexpected

At the moment, there is no difference (`toJSON` returns `$currentData`). But one of the features of Lucid that I want to implement is field serializers. `toJSON` would call the serializers.

I think this deserves `changes` entries in the docs?

No, there's a separate loaders worker for each worker thread.

@mcollina how about we make it the default in a subsequent PR where only that bit can be discussed?

/cc @addaleax I think this might impact you

I don't know if we have coverage for the REPL in CITGM, but here's a run: https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/2727/

> > Given that it is going to break usages on performance.now() without receivers, I'm marking this as a breaking change, hence a semver-major patch. > > Can you articulate...