Skip to content
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

doc: add esm examples to node:trace_events #56514

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mfdebian
Copy link
Contributor

@mfdebian mfdebian commented Jan 8, 2025

This PR adds the missing ESM counterparts of the CJS examples for the Trace events documentation.

I also destructured the (require) imports for the current CJS examples following a feedback I had received in previous PRs, except for the last one where I didn't see it necessary, but tell me what you think and I can do it quickly!

I've tested every single example and they all work/behave as expected.

Happy new year! 🥳

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Jan 8, 2025
@lpinca lpinca added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. doc Issues and PRs related to the documentations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants