Avram Tudor

Results 12 comments of Avram Tudor

Please test this before merging

@jjasghar Thank you for this, please update your commit message to match the style of the other commits in this repo

@suchislife801 do you need to handle ping responses? or is it implemented in the ws implementation?

Webpack 5.57.1 upgrade is completed on master, however, in order to benefit of xxhash64, [experiments.futureDefaults should be enabled](https://webpack.js.org/configuration/output/#outputhashfunction), which is not the case presently. I'm thinking we should go ahead...

an older commit on master i causing that

@garronej I looked over the changes, everything looks good from my side. Once you rebase and resolve the new conflicts, I'd be happy to merge this. Thank you

@suzain9084 you added some unused imports and changed spacings. Please make sure you run the linter. Also, ensure the functionality is implemented for mobile as well, since you're changing files...

@suzain9084 what is your system prompt?

@suzain9084 don't worry about it, don't create a new branch please