-
Notifications
You must be signed in to change notification settings - Fork 20
Update dependencies (NPM and others) #451
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
base: staging
Are you sure you want to change the base?
Conversation
Run `npm update` in every package to update the dependencies. Fixes a few security audit warnings.
Run our update-refs scripts to bump all non-nodejs dependencies.
Update the run-test script based on the latest vscode-test sample.
The devtool modify command is now even slower, probably due to the reduced bandwidth for robots after the AI boom.
b6ce4ad to
3935c28
Compare
e249b4c to
1d37a65
Compare
|
@WilsonZiweiWang from my experience, some tests failed due to calling Maybe I'm wrong and the tests fail for a reason, but that's my thoughts so far. |
When the integration tests are skipped, the CI can consistently pass if a little more time is given to the unit tests, for example, 10 minutes of timeout. Successful runs with integration tests skipped: According to the logs from a previous CI run and my local runs, they are all stuck at the |
|
Maybe we need to disable some test parallelization option that is new or reworded? |
The devtool commands in the scanner tests have been making the CI run unreliable. Skipping for now to proceed other PRs.
66022a9 to
d9cc7eb
Compare

Run
npm updatein every package to update the dependencies. Fixes a few security audit warnings.