Skip to content

update dependency version #2172

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

Closed
wants to merge 7 commits into from

Conversation

kazrael2119
Copy link
Member

fix #2167
update sinon, directory-tree puppeteer versions to fix this issue

@@ -1001,7 +1001,7 @@ describe("LRO Rest Client", () => {
assert.equal(result.status, "200");
});

it("should handle put200InvalidJson", async () => {
it.only("should handle put200InvalidJson", async () => {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image
repro ci test failure in my local env.

step:

  1. start a git bash
  2. run rush update && rush build
  3. cd package/autorest.typescript
  4. run rushx start-test-server:v2
  5. start a new git bash again
  6. cd package/autorest.typescript
  7. run rushx rlc-integration-test:alone
  8. run rushx stop-test-server

@@ -109,10 +109,10 @@
"node-cmd": "^3.0.0",
"npm-run-all": "^4.1.5",
"path-browserify": "^1.0.1",
"puppeteer": "^3.3.0",
"puppeteer": "^21.6.0",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you check on this.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in main, it's "^21.0.0"

@qiaozha qiaozha added the P1 priority 1 label Feb 28, 2024
@qiaozha
Copy link
Member

qiaozha commented Mar 15, 2024

can we close this as of #2357 ?

@qiaozha
Copy link
Member

qiaozha commented Mar 20, 2024

close it for now

@qiaozha qiaozha closed this Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 priority 1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

upgrade deprecated dependencies
3 participants