-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
docs: use ESM code in examples #9427
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ |
Is this the only file that needs this change? |
Can you update these too? discord.js/packages/discord.js/README.md Lines 59 to 103 in a2ab2b8
|
If you're updating that part of the README, it should probably align with my changes to the command deployment in the guide: https://github.com/discordjs/discord.js/pull/9385/files#diff-af7cad8eaebbb884ba861e35ce70fd37dcb6d78b8a491cd986356971b29fd4b9 Not the same level of detail, but use top-level-await, the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sweet TLA.
* docs(Partials): use import statement * docs: update readme to ESM --------- Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
Please describe the changes this PR makes and why it should be merged:
This PR converts the documentation examples from CommonJS to ESM.
Status and versioning classification: