Michael Lazarski comments

Results 16 comments of


                                            Michael Lazarski

feat(ligatures): adds a proposal to add ligatures

Waiting for this too.

adjustTypesToJSONValue will crash server when passed a very long string

Looking at the EJSON clone function, there are several `Object.keys`. https://github.com/meteor/meteor/blob/24865b28a0689de8b4949fb69ea1f95da647cd7a/packages/ejson/ejson.js#L174 This is called way before the mentioned `adjustTypesToJSONValue`. ```js Object.keys('a'.repeat(16*1024*1024+1)) ``` This will run out of memory. The same...

adjustTypesToJSONValue will crash server when passed a very long string

There is an PR for improving EJSON https://github.com/meteor/meteor/pull/10422

describe.each and test.each are shown as 'test result not found'

@zxch3n thanks for the info. Are you accepting PR's? If you could just point me to the right file, I could take a look at it :)

describe.each and test.each are shown as 'test result not found'

Thanks for the info, I have investigated this a little and looked into the vitest/ui package because there it works. They are basically connecting to the server like this: https://github.com/vitest-dev/vitest/blob/main/packages/ui/client/composables/client.ts...

Issues with peer dependencies

@Kiyozz I'm still getting the error after running: ``` pnpm store prune rm -rf ~/.pnpm-state ~/.pnpm-store pnpm prune rm pnpm-lock.yaml rm -rf node_modules pnpm up ```

Issues with peer dependencies

@Kiyozz nope just a project

Issues with peer dependencies

with pnpm 7.X in the package.json this works for me but only with `pnpm`. ```json "pnpm": { "peerDependencyRules": { "ignoreMissing": [ "@algolia/client-search", "react", "react-dom", "@types/react" ] } } ```

vue-apexcharts height option doesent work while height prop works

Also setting this via the `responsive` option does not work. The height is also ignored.

`z` not completing insensitively anymore

> > can you elaborate your technique?? > > as m facing this too... > > So of course this is just temporary fix as I don't think the dev...