Skip to content

Conversation

@jeremymeng
Copy link
Member

@jeremymengjeremymeng 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.
@jeremymengjeremymengforce-pushed the identity/mac-nodejs-21.2.0 branch from 002821e to d120796CompareNovember 30, 2023 23:18
@jeremymengjeremymeng changed the title [test] identity on nodejs 21.2.0 MacOSTemporarily pin NodeTestVersion of v21 to 21.2.0Nov 30, 2023
@jeremymengjeremymeng 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

@weshaggardweshaggard 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
MemberAuthor

Lets be sure to file a follow-up issue.

#27957

@jeremymengjeremymeng merged commit b1a31b3 into Azure:mainDec 1, 2023
@jeremymengjeremymeng 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 freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants

@jeremymeng@MuriloKakazu@weshaggard