Skip to content

Conversation

@dsanders11
Copy link
Member

This clears the audit. The new version of mocha requires Node.js >= 14 so this also drops Node.js 12 from our CI matrix - we'll be doing the Node.js minimum bump soon enough.

@dsanders11 dsanders11 marked this pull request as ready for review January 16, 2025 05:05
@dsanders11 dsanders11 requested a review from a team as a code owner January 16, 2025 05:05

export async function cleanupTestModule(testModulePath: string): Promise<void> {
await fs.rmdir(testModulePath, { recursive: true, maxRetries: 10 });
await fs.rm(testModulePath, { recursive: true, maxRetries: 10 });
Copy link
Member Author

Choose a reason for hiding this comment

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

Clears a warning that was showing up during test runs about fs.rmdir({ recursive: true }) being deprecated.

@dsanders11 dsanders11 merged commit 8e8e6f9 into main Jan 16, 2025
14 checks passed
@dsanders11 dsanders11 deleted the build/bump-mocha branch January 16, 2025 08:28
@continuous-auth
Copy link

🎉 This PR is included in version 3.7.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants