Skip to content

Conversation

@jeremymeng
Copy link
Member

@jeremymeng jeremymeng commented Nov 30, 2023

Latest 21.3.0 contains changes that breaks writeFileSync of temp files on
MacOS. This PR unblocks js - core release pipeline while we follow up on the
NodeJS issue.

Latest 21.3.0 contains changes that breaks `writeFileSync` of temp files on
MacOS. This PR unblocks `js - core` release pipeline while we follow up on the
NodeJS issue.
@jeremymeng jeremymeng force-pushed the identity/mac-nodejs-21.2.0 branch from 002821e to d120796 Compare November 30, 2023 23:18
@jeremymeng jeremymeng changed the title [test] identity on nodejs 21.2.0 MacOS Temporarily pin NodeTestVersion of v21 to 21.2.0 Nov 30, 2023
@jeremymeng jeremymeng marked this pull request as ready for review November 30, 2023 23:58
@MuriloKakazu
Copy link

This can be reverted when the fix is released: nodejs/node#50990

Alternatively, you can pass flag: 'w' to your writeFileSync calls so it works as expected

Copy link
Member

@weshaggard weshaggard left a comment

Choose a reason for hiding this comment

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

Lets be sure to file a follow-up issue.

@jeremymeng
Copy link
Member Author

Lets be sure to file a follow-up issue.

#27957

@jeremymeng jeremymeng merged commit b1a31b3 into Azure:main Dec 1, 2023
@jeremymeng jeremymeng deleted the identity/mac-nodejs-21.2.0 branch December 1, 2023 17:57
jeremymeng added a commit to jeremymeng/azure-sdk-for-js that referenced this pull request Dec 6, 2023
now that a fix to `fs.writeFileSync` has been released.

Revert "Temporarily pin NodeTestVersion of v21 to 21.2.0 (Azure#27951)"

This reverts commit b1a31b3.
jeremymeng added a commit that referenced this pull request Dec 6, 2023
now that a fix to `fs.writeFileSync` has been released.

Revert "Temporarily pin NodeTestVersion of v21 to 21.2.0 (#27951)"

This reverts commit b1a31b3.

### Issues associated with this PR
#27957
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

3 participants