-
Notifications
You must be signed in to change notification settings - Fork 406
More dependabot fixes #2425
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.
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
More dependabot fixes #2425
Conversation
of protobufjs doesn't include any breaking changes to rawproto based on the functions by it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@GnaneshKunal can you return back testcafe version (discard tests/e2e/package.json and tests/e2e/yarn.lock)?
there is an issue in testCafe 3.1.0+ DevExpress/testcafe#7886 which fails our 23 e2e tests
"electron-builder/app-builder-lib/@electron/universal/dir-compare/minimatch": "^3.0.5" | ||
"electron-builder/app-builder-lib/@electron/universal/dir-compare/minimatch": "^3.0.5", | ||
"**/semver": "^7.5.2", | ||
"rawproto/protobufjs": "^7.2.4" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this version compatible with "rawproto": "^0.7.6"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking into this. Basic RI UI test did pass. Running tests/same code from the library
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ran rawproto tests. Worked fine for protobuf ^7.2.4
Fixes